saveourtool / diktat

Strict coding standard for Kotlin and a custom set of rules for detecting code smells, code style issues and bugs
https://diktat.saveourtool.com
MIT License
516 stars 39 forks source link

Fixed bug in WRONG_DECLARATIONS_ORDER rule #1881

Closed diphtongue closed 6 months ago

diphtongue commented 6 months ago

What's done:

Closes #1865

github-actions[bot] commented 6 months ago

JUnit Tests (macOS, EnricoMi/publish-unit-test-result-action@v2)

   166 files  ±0     166 suites  ±0   6m 31s :stopwatch: + 2m 22s 1 432 tests +2  1 394 :heavy_check_mark: +2  38 :zzz: ±0  0 :x: ±0  2 811 runs  +2  2 773 :heavy_check_mark: +2  38 :zzz: ±0  0 :x: ±0 

Results for commit b9c1f531. ± Comparison against base commit de959e2f.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 6 months ago

JUnit Tests (Windows, EnricoMi/publish-unit-test-result-action@v2)

   166 files  ±0     166 suites  ±0   7m 18s :stopwatch: -7s 1 432 tests +2  1 415 :heavy_check_mark: +2  17 :zzz: ±0  0 :x: ±0  2 811 runs  +2  2 794 :heavy_check_mark: +2  17 :zzz: ±0  0 :x: ±0 

Results for commit b9c1f531. ± Comparison against base commit de959e2f.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 6 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (de959e2) 78.30% compared to head (b9c1f53) 78.29%.

Files Patch % Lines
...set/rules/chapter3/ClassLikeStructuresOrderRule.kt 66.66% 0 Missing and 2 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1881 +/- ## ============================================ - Coverage 78.30% 78.29% -0.02% - Complexity 2465 2469 +4 ============================================ Files 134 134 Lines 8699 8704 +5 Branches 2202 2206 +4 ============================================ + Hits 6812 6815 +3 Misses 905 905 - Partials 982 984 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.