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

corresponding default values for checkstyle CHECKSTYLEALLJAR and JPLAGJAR #246

Open
physikerwelt opened this issue Nov 24, 2016 · 5 comments

Comments

@physikerwelt
Copy link
Contributor

I wonder why the checkstyle jar should be placed to /home/praktomat
cf.

d.CHECKSTYLEALLJAR = '/home/praktomat/contrib/checkstyle-all-4.4.jar'

@nomeata
Copy link
Contributor

nomeata commented Nov 28, 2016

Could be anywhere, I guess, as long as the paths points there.

@physikerwelt
Copy link
Contributor Author

@nomeata is the "user" supposed to change both files defaults.js and local.py?

I guess it would be easier if defaults would point to some reasonable default locations and only one file (maybe ignored by git) would hold the user specific settings.

@nomeata
Copy link
Contributor

nomeata commented Nov 30, 2016

The user is supposed to change local.py, which can override any of the default settings.

@physikerwelt physikerwelt changed the title checkstyle jar corresponding default values for checkstyle CHECKSTYLEALLJAR and JPLAGJAR Dec 8, 2016
@ifrh
Copy link
Contributor

ifrh commented Jan 22, 2022

I vote for closing this issue, perhaps after introducing a section in README.md about files which should become changed by admins ...

@ifrh
Copy link
Contributor

ifrh commented Mar 7, 2022

Are information in README (update on 19. February 2022) now better?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants