davsclaus opened a new pull request, #23371:
URL: https://github.com/apache/camel/pull/23371
## Summary
- Skip Scalpel POM analysis when only the root `pom.xml` changes (not
`parent/pom.xml` or module-level `pom.xml` files)
- The root `pom.xml` contains build-infrastructure config (license plugin,
checkstyle, etc.) that doesn't affect module compilation or test behavior
- Without this filter, Scalpel reports every module as affected because they
all inherit from the root POM, causing CI to test the entire project
unnecessarily (as seen in PR #23364)
## Changes
- `.github/actions/incremental-build/incremental-build.sh`: Changed Scalpel
trigger condition to only fire when a subdirectory `pom.xml` changes — extracts
file paths from the diff with `sed` and checks for paths containing at least
one directory level before `pom.xml`
- `.github/CI-ARCHITECTURE.md`: Document the root `pom.xml` exclusion
behavior
## Test plan
- [x] Verified grep logic handles all edge cases:
- Root `pom.xml` only → Scalpel skipped
- `parent/pom.xml` → Scalpel runs
- Module `pom.xml` (e.g. `components/camel-kafka/pom.xml`) → Scalpel runs
- Root + module `pom.xml` → Scalpel runs
- No `pom.xml` changes → Scalpel skipped
- [ ] CI run on this PR should only test CI-related files (not every module)
_Claude Code on behalf of Claus Ibsen_
--
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]
