gnodet opened a new pull request, #26025: URL: https://github.com/apache/camel/pull/26025
## Summary _Claude Code on behalf of gnodet_ - Before Scalpel, when the `-amd` expansion of changed modules exceeded the threshold (50), the build would only test the directly changed modules. The CI comment correctly reported "Dependent modules were not tested because the total number of affected modules exceeded the threshold." - With Scalpel, detected dependent modules are added explicitly to `-pl` (merged at step 2c), **bypassing the threshold entirely**. For core module changes (`camel-api`, `camel-support`), this means ~591 modules get tested regardless of the threshold — making the comment misleading. - This fix strips dependency-detected modules (grep + Scalpel) from the build list when the threshold is exceeded, restoring the pre-Scalpel behavior. The `test-dependents` label still forces testing all dependents. ## Test plan - [x] Verify that CI changes are under `.github/` path-ignore and won't trigger a full build - [ ] Validate by opening a test PR that touches `core/camel-support` — should only test the directly changed modules, not ~591 dependents - [ ] Verify `test-dependents` label still overrides the threshold Co-Authored-By: Claude Opus 4.6 <[email protected]> -- 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]
