/sevntu.checkstyle

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

Rename checks to follow Checkstyle convention for check names #417

Closed
MEZk opened this Issue Dec 28, 2015 · 1 comment

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  1.18
2 participants
@MEZk @romani
@MEZk
Contributor

MEZk commented Dec 28, 2015

checkstyle/checkstyle#2780

Checkstyle name convension for checks names: XXXXCheck

The following classes should be renamed:
AvoidDefaultSerializableInInnerClasses
ForbidCCommentsInMethods
ReturnBooleanFromTernary
ReturnNullInsteadOfBoolean

@MEZk MEZk changed the title from Rename checks to folllow Checkstyle convesion for check names to Rename checks to follow Checkstyle convesion for check names Dec 28, 2015

@MEZk MEZk changed the title from Rename checks to follow Checkstyle convesion for check names to Rename checks to follow Checkstyle convension for check names Dec 28, 2015

@MEZk MEZk changed the title from Rename checks to follow Checkstyle convension for check names to Rename checks to follow Checkstyle convention for check names Dec 28, 2015

MEZk added a commit to MEZk/sevntu.checkstyle that referenced this issue Dec 28, 2015

@MEZk
Issue #417: Rename checks to follow Checkstyle convention for check n… 
…ames
4d181e9

MEZk added a commit to MEZk/sevntu.checkstyle that referenced this issue Dec 28, 2015

@MEZk
Issue #417: Rename checks to follow Checkstyle convention for check n… 
…ames
5aa7716

romani added a commit that referenced this issue Jan 1, 2016

@MEZk @romani
Issue #417: Rename checks to follow Checkstyle convention for check n… 
…ames
45e2731
@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jan 1, 2016

Member

fix is merged.
Member

romani commented Jan 1, 2016

fix is merged.

@romani romani closed this Jan 1, 2016

@romani romani added this to the 1.18 milestone Jan 1, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment