Skip to content

Fix quartz example README: correct copy-pasted description - #213

Open
Croway wants to merge 2 commits into
mainfrom
fix-quartz-readme-wording
Open

Fix quartz example README: correct copy-pasted description#213
Croway wants to merge 2 commits into
mainfrom
fix-quartz-readme-wording

Conversation

@Croway

@Croway Croway commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The quartz example README described output as "high and low numbers", which was copy-pasted from the pojo example
  • The quartz example actually logs a constant "Test" message every 5 seconds via a cron-triggered route (QuartzRoute.java), then self-terminates after one minute
  • Corrected the wording to describe the actual behavior; no code or version changes needed (pom.xml already relies solely on the parent BOM)

Test plan

  • Verified route behavior in quartz/src/main/java/sample/camel/QuartzRoute.java (cron 0/5 * * * * ?, constant "Test" body, logged via log:info)
  • Confirmed no other README inaccuracies or stray dependency versions in quartz/pom.xml

🤖 Generated with Claude Code

https://claude.ai/code/session_01HseDjsECuZhsCXxxioiD1k

Croway and others added 2 commits September 4, 2026 10:17
…edundant version

The getPetById operation depends on the publicly hosted petstore3.swagger.io
demo, which can intermittently fail (e.g. 500 errors) since it is outside
our control. Document this in the README so it isn't mistaken for a bug in
the example. Also remove the redundant explicit ${project.version} on
camel-rest-openapi-starter, since it is already managed by the Camel BOM.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HseDjsECuZhsCXxxioiD1k
The README described "high and low numbers" output, copy-pasted from
the pojo example. The quartz example actually logs a constant "Test"
message every 5 seconds via a cron-triggered route.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HseDjsECuZhsCXxxioiD1k
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant