l3r8yJ / oop-cop

The static analysis tool and a Maven plugin that will help you model your objects, classes, methods properly by rejecting your non-perfect code.
https://www.l3r8y.ru/oop-cop/
MIT License
9 stars 5 forks source link

144 #145

Closed l3r8yJ closed 6 months ago

l3r8yJ commented 6 months ago

closes #144


PR-Codex overview

This PR updates the year in the copyright notice to 2024 and adjusts the @since version in VeryLongClassName.java.

Detailed summary

The following files were skipped due to too many changes: src/test/java/ru/l3r8y/extensions/ErNamedClass.java, src/test/java/ru/l3r8y/extensions/InvalidClass.java, src/test/java/ru/l3r8y/extensions/package-info.java, src/test/java/ru/l3r8y/parser/ClassMethodsTest.java, src/main/java/ru/l3r8y/checks/MutableStateCheck.java, src/main/java/ru/l3r8y/checks/CompositeClassName.java, src/main/java/ru/l3r8y/checks/LongClassNameCheck.java, src/test/java/ru/l3r8y/extensions/LongNamedClass.java, src/main/java/ru/l3r8y/complaint/WrongClassNaming.java, src/test/java/ru/l3r8y/checks/AssignmentCheckTest.java, src/test/java/ru/l3r8y/extensions/CaseWithoutThis.java, src/test/java/ru/l3r8y/checks/CompositeErNamedTest.java, src/test/java/ru/l3r8y/complaint/BulkComplaintTest.java, src/test/java/ru/l3r8y/extensions/ManySuppressions.java, src/test/java/ru/l3r8y/checks/CompositeClassNameTest.java, src/main/java/ru/l3r8y/complaint/WrongMethodSignature.java, src/test/java/ru/l3r8y/extensions/IsSuppressedErSuffix.java, .github/workflows/xcop.yaml, src/main/java/ru/l3r8y/parser/Code.java, src/main/java/ru/l3r8y/parser/CodeClass.java, src/test/java/ru/l3r8y/parser/DefaultTest.java, src/test/java/ru/l3r8y/parser/DeclarationTest.java, src/main/java/ru/l3r8y/parser/Default.java, src/test/java/ru/l3r8y/complaint/LinkedComplaintTest.java, src/test/java/ru/l3r8y/complaint/ClassifiedComplaintTest.java, src/main/java/ru/l3r8y/parser/Declaration.java, src/main/java/ru/l3r8y/complaint/ClassifiedComplaint.java, src/test/java/ru/l3r8y/extensions/ParserDeclaration.java, src/main/java/ru/l3r8y/complaint/LinkedComplaint.java, src/test/java/ru/l3r8y/extensions/SuppressedLongClassName.java, src/test/java/ru/l3r8y/extensions/ManySuppressionsDeclaration.java, src/main/java/ru/l3r8y/parser/ClassMethods.java, src/main/java/ru/l3r8y/checks/ClassCanSuppress.java, src/main/java/ru/l3r8y/parser/IsSuppressed.java, src/test/java/ru/l3r8y/checks/ClassCanSuppressTest.java, src/main/java/ru/l3r8y/parser/IgnoresSuppressed.java, src/it/settings.xml, src/it/correct/pom.xml, src/test/java/ru/l3r8y/parser/IgnoresSuppressedTest.java, src/test/java/ru/l3r8y/parser/IsSuppressedTest.java, src/test/java/ru/l3r8y/parser/ScannedChecksTest.java, src/main/java/ru/l3r8y/parser/SuppressedChecks.java, src/test/java/ru/l3r8y/parser/SuppressedChecksTest.java, src/main/java/ru/l3r8y/parser/ScannedChecks.java, src/it/incorrect/pom.xml, src/it/long-class-name/pom.xml, pom.xml

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

zoeself commented 6 months ago

@l3r8yJ thank you for your Pull Request. I'll assign someone to review it soon.

If this PR solves a todo from the code, please don't forget to remove it.

codecov-commenter commented 6 months ago

Codecov Report

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

Comparison is base (43e030a) 81.67% compared to head (44c0a2d) 81.41%.

Files Patch % Lines
src/main/java/ru/l3r8y/parser/ScannedChecks.java 80.00% 2 Missing :warning:
...rc/main/java/ru/l3r8y/parser/SuppressedChecks.java 75.00% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #145 +/- ## ============================================ - Coverage 81.67% 81.41% -0.27% + Complexity 95 94 -1 ============================================ Files 25 25 Lines 262 269 +7 Branches 10 10 ============================================ + Hits 214 219 +5 - Misses 42 44 +2 Partials 6 6 ```

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

zoeself commented 6 months ago

@l3r8yJ I couldn't find any assignee for this task. This is either because there are no contributors with role REV available or because the project does not have enough funds.

Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.

sonarcloud[bot] commented 6 months ago

Quality Gate Passed Quality Gate passed

Issues
2 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

l3r8yJ commented 6 months ago

@rultor merge

rultor commented 6 months ago

@rultor merge

@l3r8yj OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 6 months ago

@rultor merge

@l3r8yj Done! FYI, the full log is here (took me 3min)