-
Notifications
You must be signed in to change notification settings - Fork 249
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
Add checkstyle plugin #1551
Add checkstyle plugin #1551
Conversation
Diff Coverage for cli: The code coverage on the diff in this pull request is 100.0%. Total Coverage for cli: This PR will not change total coverage. File Coverage Changes
🛟 Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
One thing of note is checkfile needs a config file to run so set to
suggested = "config"
.In the docs example the config file in most places is just
config.xml
which seems to generic, so I put it ascheckstyle.xml
for now.