gnodet opened a new pull request, #22247: URL: https://github.com/apache/camel/pull/22247
## Summary - Merge the separate `detect-dependencies` action into `incremental-build` for a single unified CI test mechanism - For dependency version bumps, the old setup produced two separate PR comments and wasted time testing `parent` (which has no tests) — now there's one comment and one test run - Generalize POM dependency analysis from parent-only to any changed `pom.xml` - Raise testable modules threshold to 1000 (effectively disabled, can be tuned later) - Remove unused `build` mode (builds are handled by `regen.sh`) ## Details Previously two independent mechanisms determined which modules to test: 1. **incremental-build**: file-path based module detection — good for source changes, but for `parent/pom.xml` changes it would identify `parent`, find everything depends on it, and give up 2. **detect-dependencies**: POM property change analysis using Maveniverse Toolbox — accurately found the 3 modules affected by a version bump, but ran separately The merged script combines both approaches: file-path analysis for source changes, Toolbox dependency analysis for pom.xml property changes. Results are merged, deduplicated, and tested once. ## Test plan - [ ] Verify CI runs correctly on this PR (the `.github/**` paths-ignore will skip it, so may need manual trigger) - [ ] Test with a dependency bump PR to verify Toolbox detection works end-to-end - [ ] Test with a component source change PR to verify file-path detection still works -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
