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

checkstyle bear #1067

Closed
sils opened this issue Nov 18, 2015 · 5 comments
Closed

checkstyle bear #1067

sils opened this issue Nov 18, 2015 · 5 comments

Comments

@sils
Copy link
Member

sils commented Nov 18, 2015

java checkstyle

@rubdos
Copy link

rubdos commented Feb 1, 2016

wget "http://iweb.dl.sourceforge.net/project/checkstyle/checkstyle/6.15/checkstyle-6.15-all.jar"
wget "https://raw.githubusercontent.com/checkstyle/checkstyle/master/src/main/resources/google_checks.xml"
java -jar checkstyle-6.15-all.jar -c google_checks.xml ClassName.java

@sils
Copy link
Member Author

sils commented Feb 1, 2016

@rubdos thanks. While thinking about it there's one unsolved issue still - we need to install checkstyle somehow because we don't want the execution to be dependent on where the user runs coala, right?

sils added a commit that referenced this issue Feb 1, 2016
TODO: Unittests, install checkstyle on CI, install checkstyle globally

Fixes #1067
@rubdos
Copy link

rubdos commented Feb 1, 2016

@sils1297 You probably want to have the user/coala put it in some standard location or something.

@rubdos
Copy link

rubdos commented Feb 2, 2016

I found: https://oss.sonatype.org/content/repositories/snapshots/com/puppycrawl/tools/checkstyle/
which is their official maven snapshot repo
and http://repo1.maven.org/maven2/com/puppycrawl/tools/checkstyle/
which is their release repo.

Looks like file are persistent there, so this could be something.

sils added a commit that referenced this issue Feb 3, 2016
TODO: Unittests, install checkstyle on CI, install checkstyle globally

Fixes #1067
sils added a commit that referenced this issue Feb 5, 2016
TODO: Unittests, install checkstyle on CI, install checkstyle globally

Fixes #1067
@rultor rultor closed this as completed in 9bea8fe Feb 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants