chore: apply checkstyle again#1339
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1339 +/- ##
============================================
- Coverage 78.78% 78.57% -0.22%
Complexity 651 651
============================================
Files 74 74
Lines 2498 2506 +8
Branches 259 259
============================================
+ Hits 1968 1969 +1
- Misses 448 455 +7
Partials 82 82 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
LGTM. I ran Checkstyle on this branch, and there were no other changes triggered. |
|
Sonarqube reliability lint in examples - I have renamed the sonarqube config file on the batch branch so that we exclude examples from the copy-paste analysis. You might like to do the same here to get past the quality gate. The previous config file name was incorrect for the analysis mode we are using and had no effect. |
|
Kudos, SonarCloud Quality Gate passed!
|








Issue #, if available:
Description of changes:
Looks like the latest version of Intellij or the checkstyle plugin for intellij does not format the same way than before (when I pushed my checkstyle code I was on a previous version of IntelliJ).
And looks like some formatted code has been unformatted since I pushed (bad merges or something).
So I apply it again.
@scottgerring @mriccia @eldimi @kozub @msailes @AlexeySoshin, please make sure to have checkstyle plugin installed with the latest version of IntelliJ and apply it to your changes (not to the whole project). See contributing guide. When merging main into your branches, please make sure to apply the formatting changes too.
Checklist
Breaking change checklist
RFC issue #:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.