Skip to content

[DO NOT MERGE] Test: build WITHOUT OpenRewrite (no FQCNs)#23068

Closed
gnodet wants to merge 2 commits into
apache:mainfrom
gnodet:test-no-fqcn
Closed

[DO NOT MERGE] Test: build WITHOUT OpenRewrite (no FQCNs)#23068
gnodet wants to merge 2 commits into
apache:mainfrom
gnodet:test-no-fqcn

Conversation

@gnodet
Copy link
Copy Markdown
Contributor

@gnodet gnodet commented May 7, 2026

Test PR to measure CI build time when OpenRewrite is not triggered.

Contains a trivial Javadoc change with no FQCNs — the regen.sh check should skip OpenRewrite.

Companion to the "with FQCN" test PR for comparison.

DO NOT MERGE — close after measuring build times.

gnodet and others added 2 commits May 7, 2026 17:05
Instead of always running OpenRewrite (which quadruples build time),
detect FQCNs in the diff and only enable -Prewrite when needed.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
DO NOT MERGE — test PR to measure build time without OpenRewrite.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

🌟 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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

🧪 CI tested the following changed modules:

  • core/camel-api
  • etc

ℹ️ 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.

Build reactor — dependencies compiled but only changed modules were tested (2 modules)
  • Camel :: API
  • Camel :: Etc

⚙️ View full build and test results

@gnodet
Copy link
Copy Markdown
Contributor Author

gnodet commented May 7, 2026

Claude Code on behalf of Guillaume Nodet

Closing test PR — results captured in #23060.

@gnodet gnodet closed this May 7, 2026
@gnodet gnodet deleted the test-no-fqcn branch May 7, 2026 21:02
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