Checkstyle

[QuintinWillison]

In lieu of #21, the Java code needs some linter love.

┆Issue is synchronized with this Jira Uncategorised by Unito

[QuintinWillison]

To add a little more detail... Lack of lint/static-analysis is causing a lot of noise right now in pull requests and I would prefer to allow reviewers to focus on the stuff that matters, and the machines (CI) to catch the rest.

Currently, checkstyle is explicitly disabled in our POM. I had a quick run, locally, at re-enabling it and the checkstyle:checkstyle report isn't pretty. 😞

Consistent formatting is always nice, but I think I'm more concerned around the lack of static analysis to catch the obvious oversights and/or risks.