Skip to content

Issue #15016: Remove underscore from illegal identifers format#15117

Merged
nrmancuso merged 1 commit into
checkstyle:masterfrom
checkstyle-GSoC25:illegalidentifier
Jun 28, 2024
Merged

Issue #15016: Remove underscore from illegal identifers format#15117
nrmancuso merged 1 commit into
checkstyle:masterfrom
checkstyle-GSoC25:illegalidentifier

Conversation

@mahfouz72

Copy link
Copy Markdown
Member Author

Github, generate site

* that are restricted or contextual keywords. Examples include "yield", "record", and
* "var". Please read more at
* <a href="https://docs.oracle.com/javase/specs/jls/se14/html/jls-3.html#jls-3.9">
* <a href="https://docs.oracle.com/javase/specs/jls/se22/html/jls-3.html#jls-3.9">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mahfouz72 @nrmancuso What do you guys think of making this an issue? Finding all old JLS and moving them to 22.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea I am ok with this it can be a separate issue to keep tacker of all checks or I can update it in other PRs (normal updates) whenever I see an old link in the doc
I think a separate issue would be better

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But does updating links add value? I think most of the links either 22 or 14 will have no difference in the content. there will be a difference ofc in links related to the new syntax (patterns and switch)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, new issue. Value is ensuring we have no old document links. I am not saying it is a must right now.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rnveach you can create an issue for this, but we will not cover it during GSOC.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@rnveach rnveach assigned nrmancuso and unassigned nrmancuso Jun 26, 2024
@mahfouz72 mahfouz72 force-pushed the illegalidentifier branch 2 times, most recently from 0a14aba to b3190d5 Compare June 27, 2024 11:02
@mahfouz72 mahfouz72 force-pushed the illegalidentifier branch 2 times, most recently from 177cdea to 9dbfe23 Compare June 27, 2024 11:17
@rnveach rnveach assigned nrmancuso and unassigned rnveach Jun 27, 2024
@rnveach

rnveach commented Jun 27, 2024

Copy link
Copy Markdown
Contributor

@nrmancuso Please see #15117 (comment) and merge this when you are ready.

@nrmancuso nrmancuso merged commit f2594a1 into checkstyle:master Jun 28, 2024
@mahfouz72 mahfouz72 deleted the illegalidentifier branch May 9, 2025 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

Add Check Support for Java 21 Unnamed Variables & Patterns Syntax: IllegalIdentifierName

3 participants