gnodet commented on PR #27841: URL: https://github.com/apache/camel/pull/27841#issuecomment-6214509072
All review findings from @davsclaus have been addressed across the subsequent commits: - Unchecked lines now read as `checked=false` / `unchecked` count (not rejected) - Group matching is dash- and case-insensitive (`camel.error-registry.*` etc.) - Spring Boot false positives downgraded to `checked=false` via `onlyUnknownOptionErrors` - `mainModel()` is skipped for `camel.component.*`, `camel.dataformat.*`, `camel.language.*` keys - Dead code and `matchMainModelGroup` prefix bug fixed - All new tests pass: `testValidateConfigurationPropertyNestedGroup`, `testValidateConfigurationPropertyDashedGroupName`, `testValidateConfigurationPropertyServerInventedOption`, `springBootStarterOnlyOptionIsUnchecked` CI is green. All review threads are resolved. PTAL @davsclaus for a re-review when you get a chance. -- 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]
