SONARJAVA-6093 Implement S3051 checking the signature of the main method.#5442
Open
tomasz-tylenda-sonarsource wants to merge 2 commits intomasterfrom
Open
SONARJAVA-6093 Implement S3051 checking the signature of the main method.#5442tomasz-tylenda-sonarsource wants to merge 2 commits intomasterfrom
tomasz-tylenda-sonarsource wants to merge 2 commits intomasterfrom
Conversation
tomasz-tylenda-sonarsource
force-pushed
the
tt/SONARJAVA-6093_S3051_add_j25_main_signatures
branch
3 times, most recently
from
e8d2324 to
e65c0da
Compare
tomasz-tylenda-sonarsource
force-pushed
the
tt/SONARJAVA-6093_S3051_add_j25_main_signatures
branch
3 times, most recently
from
9984aa4 to
3b02f00
Compare
tomasz-tylenda-sonarsource
force-pushed
the
tt/SONARJAVA-6093_S3051_add_j25_main_signatures
branch
from
3b02f00 to
fbeb1d9
Compare
|
rombirli
approved these changes
Feb 12, 2026
Contributor
rombirli
left a comment
rombirli
left a comment
There was a problem hiding this comment.
LGTM, good catch to test the records enums and interfaces
| public class MainJava21 { | ||
| public static void main(String[] args) { // Compliant | ||
| } | ||
|
|
Contributor
There was a problem hiding this comment.
Could we verify a non-compliant main(int args) signature against Java 21? Just to ensure we dont break anything on this side
| @@ -0,0 +1,35 @@ | |||
| <h2>Why is this an issue?</h2> | |||
| <p>"A rose by any other name would smell as sweet," but <code>main</code> by any other name would not. Just because a method has the name "main", that | |||
Contributor
There was a problem hiding this comment.
I like this creative description
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.