Skip to content

Apply UnnecessarySemicolonInEnumeration Checkstyle module - #408

Closed
izeye wants to merge 1 commit into
spring-io:mainfrom
izeye:UnnecessarySemicolonInEnumeration
Closed

Apply UnnecessarySemicolonInEnumeration Checkstyle module#408
izeye wants to merge 1 commit into
spring-io:mainfrom
izeye:UnnecessarySemicolonInEnumeration

Conversation

@izeye

Copy link
Copy Markdown
Contributor

This PR applies the UnnecessarySemicolonInEnumeration Checkstyle module.

This PR also applies it to this project itself and fixes its violations.

See spring-projects/spring-framework#32514 and spring-projects/spring-boot@9cdd0c3

@philwebbphilwebb added this to the 0.0.42 milestone Mar 25, 2024
<module name="com.puppycrawl.tools.checkstyle.checks.UpperEllCheck" />
<module name="com.puppycrawl.tools.checkstyle.checks.ArrayTypeStyleCheck" />
<module name="com.puppycrawl.tools.checkstyle.checks.OuterTypeFilenameCheck" />
<module name="UnnecessarySemicolonInEnumeration"/>

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.

Note for when we merge this: for consistency, the name should be fully-qualified:

com.puppycrawl.tools.checkstyle.checks.coding.UnnecessarySemicolonInEnumerationCheck

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

@wilkinsona Thanks! I updated it as you suggested.

@izeye
izeyeforce-pushed the UnnecessarySemicolonInEnumeration branch from 723779f to a62f004CompareMarch 27, 2024 23:58
This commit also applies it to this project itself and fixes its violations.
@izeye
izeyeforce-pushed the UnnecessarySemicolonInEnumeration branch from a62f004 to a52923fCompareMarch 28, 2024 00:05
@wilkinsonawilkinsona self-assigned this Mar 28, 2024
wilkinsona pushed a commit that referenced this pull request Mar 28, 2024
This commit also applies it to this project itself and fixes its violations.
See gh-408
@izeye
izeye deleted the UnnecessarySemicolonInEnumeration branch March 28, 2024 14:05
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@izeye@wilkinsona@philwebb