Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Checkstyle to 10.18.2 #2847

Merged
merged 3 commits into from
Oct 19, 2024
Merged

Update Checkstyle to 10.18.2 #2847

merged 3 commits into from
Oct 19, 2024

Conversation

sfeilmeier
Copy link
Contributor

See https://github.com/checkstyle/checkstyle/releases/tag/checkstyle-10.18.2 for details.

Unfortunately Checkstyle by default requires no newline after <p> in JavaDoc, whereas Eclipse IDE automatically adds one by default. This is PR disables this check.

Fixes Checstyle `Checks that overloaded methods are grouped together. Overloaded methods have the samename but different signatures where the signature can differ by the number ofinput parameters or type of input parameters or both. `
Copy link

codecov bot commented Oct 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #2847   +/-   ##
==========================================
  Coverage      56.40%   56.40%           
  Complexity      8667     8667           
==========================================
  Files           2147     2147           
  Lines          90682    90682           
  Branches        6746     6746           
==========================================
  Hits           51136    51136           
  Misses         37721    37721           
  Partials        1825     1825           

@sfeilmeier sfeilmeier merged commit 25febfa into develop Oct 19, 2024
7 checks passed
@sfeilmeier sfeilmeier deleted the feature/update-checkstyle branch October 19, 2024 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant