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

Issue #41: updated pmd validation #93

Merged

rnveach merged 2 commits into checkstyle:master from rnveach:issue_41_4

Nov 15, 2017

Member

rnveach commented Nov 15, 2017

Issue #41

rnveach added 2 commits

November 14, 2017 20:56


          Issue checkstyle#41: updated pmd validation

8fae99e


          Issue checkstyle#41: fixed pmd violations

c3391bd

Member Author

rnveach commented Nov 15, 2017

I am going to merge this but travis failure looks like regression was set to too big a change and is causing out of memory, but I don't know why the PR changing it didn't fail then.
From logs, it looks like it still did regression on old commits as list of modules don't match the commits but I don't know why.
https://travis-ci.org/checkstyle/regression-tool/jobs/302267234#L1024
https://travis-ci.org/checkstyle/regression-tool/jobs/302097214 (must download logs to see)

rnveach merged commit 702e520 into checkstyle:master

rnveach deleted the issue_41_4 branch

November 15, 2017 02:15

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

Labels

None yet