Remove Checkstyle @SuppressWarnings, disable for tests

[fsteeg]

Will resolve #62.

[blackwinter]

Just had a quick look. 1. I assumed we would only disable specific checks for the tests. 2. The Checkstyle config for the annotation should be removed if it's no longer used.

[fsteeg]

1. I assumed we would only disable specific checks for the tests.

I did look into that again yesterday, but there seems to be no straightforward way to do that. One option would be to have separate configs for main and test, but that would duplicate a lot. As mentioned in #62 (comment) and in our meeting, I think that was my reason to introduce annotation support, as it provides a convenient way to disable specific checks for a file / class. But I'm not picky here, it would all be fine with me: duplicate / separate configs, off for all tests (current state), or some proper setup in Checkstyle that I didn't find / get.

2. The Checkstyle config for the annotation should be removed if it's no longer used.

Right, thanks. After removing it, I also found missing annotations that didn't match my simple search pattern (multiple checks disabled). Pushed both in 632802a.

[blackwinter]

You should be able to specify multiple `SuppressionSingleFilter`s with different `checks` for different `files`.

[fsteeg]

Oh, right, thanks, how did I not find that? Pushed in ea789ff.

[blackwinter]

👍