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.

Sign up for GitHub

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

Jump to bottom

update example of AST for TokenTypes.LPAREN #9510

Closed
pbludov opened this issue Mar 14, 2021 · 2 comments · Fixed by #9851
Closed

update example of AST for TokenTypes.LPAREN #9510

pbludov opened this issue Mar 14, 2021 · 2 comments · Fixed by #9851
Labels
approved good first issue miscellaneous
Milestone
8.42

Comments

@pbludov
Copy link
Member

pbludov commented Mar 14, 2021

https://checkstyle.org/apidocs/com/puppycrawl/tools/checkstyle/api/TokenTypes.html#LPAREN

needs to be extended to show example of AST.
Source to be updated: https://github.com/checkstyle/checkstyle/blob/master/src/main/java/com/puppycrawl/tools/checkstyle/api/TokenTypes.java

PLEASE READ for more details: https://github.com/checkstyle/checkstyle/wiki/Good-practice-on-how-to-update-AST-in-TokenTypes
@stonefix
Copy link
Contributor

stonefix commented Apr 3, 2021

I am working on it.

@stonefix stonefix mentioned this issue Apr 5, 2021
Closed
stonefix added a commit to stonefix/checkstyle that referenced this issue Apr 11, 2021
@stonefix
Issue checkstyle#9510: update example of AST for TokenTypes.LPAREN
6c31bca
@stonefix stonefix mentioned this issue Apr 11, 2021
Merged
romani pushed a commit that referenced this issue Apr 11, 2021
@stonefix @romani
Issue #9510: update example of AST for TokenTypes.LPAREN
9f16b2a
@romani romani added this to the 8.42 milestone Apr 11, 2021
@romani
Copy link
Member

romani commented Apr 11, 2021

merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
approved good first issue miscellaneous
Projects
None yet
Milestone
8.42
3 participants
@romani @pbludov @stonefix