Skip to content

ci: test Scalpel 0.3.3 shadow — jackson2 [DO NOT MERGE]#23172

Closed
gnodet wants to merge 9 commits into
mainfrom
test-scalpel-pom-c
Closed

ci: test Scalpel 0.3.3 shadow — jackson2 [DO NOT MERGE]#23172
gnodet wants to merge 9 commits into
mainfrom
test-scalpel-pom-c

Conversation

@gnodet
Copy link
Copy Markdown
Contributor

@gnodet gnodet commented May 12, 2026

Test PR for Scalpel 0.3.3 shadow comparison. Changes jackson2-version 2.21.3→2.21.2. DO NOT MERGE.

gnodet and others added 9 commits May 7, 2026 16:01
Upgrade Scalpel from extension3:0.1.0 to extension:0.3.0 and run it in
shadow mode — observing what skip-tests mode would do without affecting
actual test execution. This allows validating Scalpel's decisions across
many PRs before switching to Scalpel-driven test execution.

Changes:
- Upgrade Scalpel to 0.3.0 with source-set-aware propagation and
  skipTestsForDownstreamModules support
- Pre-fetch base branch in CI workflow (avoids JGit issues in shallow clones)
- Remove persist-credentials: false (needed for git fetch operations)
- Add shadow comparison section to PR comments showing what Scalpel
  skip-tests mode would have tested vs the current approach
- Update CI-ARCHITECTURE.md with shadow comparison documentation

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Scalpel 0.3.0 upgrade is already on main. This PR focuses on the shadow
comparison feature, not the version bump.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Switch the grep-based incremental build script from fetching diffs via
the GitHub REST API to using local git merge-base + diff. Both the
grep script and Scalpel now share the same --deepen=200 fetch step,
removing the GitHub API dependency for diff fetching.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@gnodet gnodet closed this May 13, 2026
@gnodet gnodet deleted the test-scalpel-pom-c branch May 13, 2026 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant