Skip to content

Conversation

@optionsome
Copy link
Member

Summary

Adds checkstyle rules that are possible to break (i.e. they are not handled by prettier) that we are not currently breaking. Therefore, this pr contains no code changes. This doesn't really slow down checkstyle processing.

Issue

Relates to #7068

Unit tests

Not needed

Documentation

No

Changelog

Skipped

@optionsome optionsome added this to the 2.9 (next release) milestone Jan 27, 2026
@optionsome optionsome requested a review from a team as a code owner January 27, 2026 10:45
@optionsome optionsome added !Technical Debt Improve code quality, no functional changes. +Skip Changelog This is not a relevant change for a product owner since last release. labels Jan 27, 2026
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.14%. Comparing base (f179e2b) to head (c5de731).
⚠️ Report is 16 commits behind head on dev-2.x.

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #7244      +/-   ##
=============================================
- Coverage      72.15%   72.14%   -0.01%     
- Complexity     21088    21090       +2     
=============================================
  Files           2297     2297              
  Lines          85223    85223              
  Branches        8487     8487              
=============================================
- Hits           61490    61485       -5     
- Misses         20753    20763      +10     
+ Partials        2980     2975       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@t2gran t2gran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the increase in the build time when adding this?

…rser/AbstractCsvParserTest.java

Co-authored-by: Thomas Gran <[email protected]>
@optionsome
Copy link
Member Author

What is the increase in the build time when adding this?

There really isn't any (at least when you are not running openrewrite).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

+Skip Changelog This is not a relevant change for a product owner since last release. !Technical Debt Improve code quality, no functional changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants