Dhairya-007 commented on PR #29644:
URL: https://github.com/apache/camel/pull/29644#issuecomment-6802113692
Thanks — both points fixed.
**The intentional typo is reverted.** `SourceRouteValidationTest` types
`peroid=1000` on purpose and then asserts `contains("peroid=1000")`, so the
two comments must keep the misspelling they describe. Same mistake as
`QuickFixes` in #29643; codespell can't tell a typo from an example of one,
and I hadn't checked for that. I re-read the other 82 corrections on this
branch for the same shape and found no others.
**The three extra typos are in.** `SSLContextParamters` →
`SSLContextParameters`, "to much hassle" → "too much hassle", and
`inexisting` → `non-existing` — that last one appears twice in
`ElasticsearchRestClientComponentIT` (lines 168 and 240), so I did both to
keep the file consistent.
One more thing I found while checking this branch: the `ObjectFactory` typo
fix lengthened a Javadoc line past 120 columns, so the formatter would have
re-wrapped that paragraph and the build would have reported uncommitted
changes. It is now wrapped the way the formatter would write it. I checked
every Java paragraph that this PR and my other open PRs touch the same way
(by porting the 120-column wrap and calibrating it against unmodified files
on `main` until it reproduced them byte-for-byte, since `mvn` isn't available
to me); the only other hits were two paragraphs in #29647, fixed there.
--
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]
