Closed
Conversation
Merge three separate CI actions (incremental-build, detect-dependencies, component-test) into one unified incremental-build script and simplify the workflow architecture. Deleted: .github/actions/component-test/ .github/actions/detect-dependencies/ Added: .github/workflows/pr-test-commenter.yml .github/CI-ARCHITECTURE.md Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Restore maxNumberOfTestableProjects to 50 (was accidentally changed to 1000) - Restore root project change handling: exit early with informative comment - Restrict POM dependency analysis to parent/pom.xml only (matches original detect-test.sh behavior; detection improvements deferred to follow-up PR) - Add explicit comment about intentional Toolbox removal (pre-apache#22022 approach) - Add skip_full_build input for /component-test dispatches to avoid full regen.sh build (uses quick targeted build with -Dquickly instead) - Document multi-JDK artifact upload behavior (intentional overwrite for resilience when one JDK fails) - Update CI-ARCHITECTURE.md to reflect all changes Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
DO NOT MERGE — will be dropped before merging to main. - Allow workflow to run on gnodet/camel fork - Add dummy parent/pom.xml change to trigger pull_request event (exercises POM dependency detection path) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
gnodet
force-pushed
the
ci-merge-incremental-build-detect-dependencies
branch
from
e2c0d71 to
d87d56a
Compare
|
ℹ️ CI did not run targeted module tests (root project files changed). |
Owner
Author
|
Claude Code on behalf of Guillaume Nodet — Closing test PR. The CI run confirmed the incremental-build.sh script works correctly. The detect-dependencies failure is expected (old workflow from main references deleted action). Full validation will happen on apache/camel PR apache#22247. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Test run for apache#22247 — verifying CI workflow changes on fork.
What this tests:
parent/pom.xml)skip_full_buildinput for component-test dispatchespr-test-commenter.ymlworkflow_run comment postingDO NOT MERGE — test-only PR, will be closed after CI verification.