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
OverloadMethodDeclarationOrder check ignores anything besides methods #14765
Comments
I'm working on it! |
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 8, 2024
3 tasks
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 9, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 9, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 11, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 14, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 16, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 20, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 20, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 20, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 20, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 20, 2024
Zopsss
added a commit
to Zopsss/checkstyle
that referenced
this issue
Apr 20, 2024
nrmancuso
pushed a commit
that referenced
this issue
Apr 28, 2024
Closed via #14766 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have read check documentation: https://checkstyle.org/checks/coding/overloadmethodsdeclarationorder.html
I have downloaded the latest checkstyle from: https://checkstyle.org/cmdline.html#Download_and_Run
I have executed the cli and showed it below, as cli describes the problem better than 1,000 words
The OverloadMethodDeclarationOrderCheck gives us error if there is another method between the overloaded methods. But it does not gives error when there is anything else besides method between overloaded methods like a variable, array, inner interface, inner class, etc.
This is how the check works currently:
Test.java
config.xml
Output
The check should be giving error for the last method
Describe what you expect in detail.
The check should be give an error something like this for the above example:
The text was updated successfully, but these errors were encountered: