gnodet opened a new pull request, #22098:
URL: https://github.com/apache/camel/pull/22098
## Summary
- When CI can't test all dependent modules (> 50 threshold), the PR comment
previously showed "Full reactor (N modules)" â making it look like all N
modules were tested when only the changed modules actually were
- Now the comment clearly indicates:
- An info note explaining dependents were skipped and how to force them
(`test-dependents` label)
- The reactor summary is labeled "Build reactor â dependencies compiled
but only changed modules were tested"
- When dependents ARE tested (< threshold or `test-dependents` label), the
label remains "All tested modules"
Example of improved comment when dependents are skipped:
> :test_tube: **CI tested the following changed modules:**
> - `components/camel-as2/camel-as2-api`
>
> > âšī¸ Dependent modules were not tested because the total number of
affected modules exceeded the threshold (50). Use the `test-dependents` label
to force testing all dependents.
## Test plan
- [x] Reviewed logic flow for all test paths (test-dependents label,
threshold exceeded, normal)
- [x] Verified `testedDependents` variable is properly initialized and set
in all branches
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]
