|
Description:
|
Checkstyle doesn't seem to catch some violations, like:
https://github.com/hibernate/hibernate-search/pull/419/files#L17L255 {code} - setSpatialParameters( Point.fromDegrees( latitude, longitude ), fieldName ); + setSpatialParameters( Point.fromDegrees( latitude, longitude ), fieldName ); {code}
or
and
{code} - for (Object o : resultList) { + for ( Object o : resultList ) { {code}
and
{code} - throw new RuntimeException( e ); + throw new RuntimeException( e ); {code} In OGM checkstyle skipped the test submodules at some point. I solved the problem upgrading the version. I write this down as a reminder of something that should be checked.
UselessShardingStrategy seems to be a good example of violations that should be found.
|