Remove/fix/change linter
Closed this issue · 5 comments
danielt998 commented
Remove/fix/change linter
james-s-w-clark commented
What's broken? What do we have, and what would we like?
danielt998 commented
it fails on every pr. We would like it not to do that
james-s-w-clark commented
https://github.com/github/super-linter#template-rules-files
We should customisesome things, like not requiring javadocs
james-s-w-clark commented
First try this:
install checkstyle plugin
and restart Intellij
If that doesn't work, try this too:
https://stackoverflow.com/questions/55248370/intellij-checkstyle-plugin-reformat
Do this with
https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/google_checks.xml
Intellij seems to now be using Google checkstyle. It will be easier to see what rules we (dis)agree with if we can see them in the IDE
james-s-w-clark commented
#105, we can close this after merge