Skip to content

Commit a5dc486

Browse files
committed
docs: add TODO for existing workflow migration
- Document existing disabled workflow at .github/workflows/integration-tests.yml - Note migration considerations and decision points - Plan to revisit after Phase 5 (Automated Triggers)
1 parent 4c95a95 commit a5dc486

File tree

1 file changed

+42
-0
lines changed

1 file changed

+42
-0
lines changed
Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
# TODO: Revisit Existing Integration Tests Workflow
2+
3+
## Context
4+
There is a pre-existing GitHub Actions workflow at `.github/workflows/integration-tests.yml` that has been disabled.
5+
6+
## Location
7+
https://github.com/spring-projects/spring-ai-examples/blob/main/.github/workflows/integration-tests.yml
8+
9+
## Tasks to Complete
10+
- [ ] Review the existing workflow structure and features
11+
- [ ] Identify what can be migrated/merged from the old workflow
12+
- [ ] Determine if both workflows should coexist or if they should be merged
13+
- [ ] Consider the following aspects:
14+
- Trigger mechanisms (push, PR, schedule)
15+
- Test coverage scope
16+
- Performance optimizations
17+
- Caching strategies
18+
- Matrix strategies
19+
- Secret management
20+
21+
## Migration Considerations
22+
- The new workflow uses the updated `run-integration-tests.sh` script
23+
- The new workflow includes Spring AI version management
24+
- The old workflow may have different test selection logic
25+
- Check for any special environment configurations
26+
27+
## Decision Points
28+
1. Keep both workflows with different purposes?
29+
- Old: Automated on push/PR
30+
- New: Manual with version control
31+
32+
2. Merge into single comprehensive workflow?
33+
- Combine triggers
34+
- Unify test selection
35+
- Consolidate configuration
36+
37+
3. Replace old with new entirely?
38+
- Migrate all features to new workflow
39+
- Archive old workflow
40+
41+
## Timeline
42+
Review after Phase 5 (Automated Triggers) is complete, as that phase will add push/PR triggers to the new workflow.

0 commit comments

Comments
 (0)