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.

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

PHOENIX-5428 Upgrade maven-checkstyle-plugin version #565

Closed
wants to merge 1 commit into from

Conversation

stoty
Copy link
Contributor

@stoty stoty commented Aug 6, 2019

This upgrades maven-checkstyle-plugin to 3.1.0 and
updates the checker.xml config files to work with recent
checkstyle versions

This upgrades maven-checkstyle-plugin to 3.1.0 and
updates the checker.xml config files to work with recent
checkstyle versions
@joshelser
Copy link
Member

Looks like we're skipping checkstyle for the build now, but I can see that this does run validations. Tough to say if the change is "OK" since there are so many violations.

asfgit pushed a commit that referenced this pull request Sep 4, 2019
This upgrades maven-checkstyle-plugin to 3.1.0 and
updates the checker.xml config files to work with recent
checkstyle versions

Closes #565

Signed-off-by: Josh Elser <elserj@apache.org>
asfgit pushed a commit that referenced this pull request Sep 4, 2019
This upgrades maven-checkstyle-plugin to 3.1.0 and
updates the checker.xml config files to work with recent
checkstyle versions

Closes #565

Signed-off-by: Josh Elser <elserj@apache.org>
asfgit pushed a commit that referenced this pull request Sep 4, 2019
This upgrades maven-checkstyle-plugin to 3.1.0 and
updates the checker.xml config files to work with recent
checkstyle versions

Closes #565

Signed-off-by: Josh Elser <elserj@apache.org>
@asfgit asfgit closed this in c60b91a Sep 4, 2019
@stoty stoty deleted the PHOENIX-5428 branch January 9, 2020 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants