danielt998/HanziToAnki

Remove/fix/change linter

Closed this issue · 5 comments

Remove/fix/change linter

What's broken? What do we have, and what would we like?

it fails on every pr. We would like it not to do that

https://github.com/github/super-linter#template-rules-files

We should customisesome things, like not requiring javadocs

First try this:
install checkstyle plugin
image
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
image

#105, we can close this after merge