gnodet opened a new pull request, #22022:
URL: https://github.com/apache/camel/pull/22022
## Summary
The `detect-dependencies` CI action was **never finding affected modules**
for dependabot PRs that bump versions in `parent/pom.xml`.
**Root cause**: The script searched for `${property-name}` (e.g.,
`${jaxb-impl-version}`) in module `pom.xml` files. This never matches because
modules inherit managed versions from the parent's `<dependencyManagement>`
without referencing the property directly — they just declare
`<artifactId>jaxb-impl</artifactId>` without a `<version>`.
**Fix**: New approach that maps properties to artifacts to modules:
1. Diff `parent/pom.xml` to find changed property names
2. Parse `parent/pom.xml` to find which `<artifactId>` uses each property as
its `<version>` in `<dependencyManagement>`
3. Search module `pom.xml` files for those artifact IDs
4. Run tests for affected modules
**Example**: For PR #22019 (`jaxb-impl-version` bump), the old script found
0 modules. The new script correctly finds `camel-drill`, `camel-ibm-cos`, and
`camel-swift`.
### Additional improvements
- Posts a PR comment summarizing which modules were tested and whether tests
passed/failed
- Writes step summary output
- Removes the overly restrictive `version|dependency|artifact` keyword
filter on property names
- Adds proper `skip-mvnd-install` input to avoid reinstalling mvnd
- Only runs on non-experimental matrix builds (skips JDK 25)
- Caps at 50 modules to avoid excessive test runs
## Test plan
- [x] Verified locally that `jaxb-impl-version` → `jaxb-impl` →
`camel-drill, camel-ibm-cos, camel-swift` detection works
- [ ] CI will validate the script runs without errors
- [ ] Next dependabot PR touching `parent/pom.xml` will exercise the full
flow
--
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]
