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

Improved ChildBlockLengthCheck a bit #97

Merged
merged 1 commit into from Jan 13, 2013

Conversation

Projects
None yet
2 participants
@daniilyar
Member

daniilyar commented Jan 13, 2013

Improved ChildBlockLengthCheck to show current size of 'bad' blocks, not only allowed sizes for it.

Also I`ve fixed the problem when Maven is unable to find system 'com.sun:tools' library (which comes with Sun Java by default and present in JAVA_HOME/lib, but is invisible for Maven).

see http://stackoverflow.com/questions/8375423/missing-artifact-com-suntoolsjar
and http://maven.apache.org/general.html - paragraph ' How do I include tools.jar in my dependencies? '

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jan 13, 2013

Member

please squash commits together, all other is ok.

Member

romani commented Jan 13, 2013

please squash commits together, all other is ok.

Daniil Yaroslavtsev
Improved ChildBlockLengthCheck to show current size of 'bad' blocks, …
…not only allowed size for it.

Minor. Removed redundant .checkstyle file
@daniilyar

This comment has been minimized.

Show comment
Hide comment
@daniilyar

daniilyar Jan 13, 2013

Member

done

Member

daniilyar commented Jan 13, 2013

done

romani added a commit that referenced this pull request Jan 13, 2013

Merge pull request #97 from daniilyar/master
Improved ChildBlockLengthCheck a bit

@romani romani merged commit b3f9e1a into sevntu-checkstyle:master Jan 13, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment