search

ObeoNetwork / POOSL

Eclipse Public License 2.0
1 stars 0 forks source link

Use stricter checkstyle configuration #6

Closed wpiers closed 2 years ago

wpiers commented 3 years ago

At this point, https://github.com/ObeoNetwork/POOSL/blob/main/parent/preferences/CheckstyleConfiguration.xml is active and in the future we need to move to https://github.com/ObeoNetwork/POOSL/blob/main/parent/preferences/CheckstyleConfiguration-strict.xml. This is the same config, it just pass some issues to the error level.

mypsycho commented 2 years ago

InnerAssignment and ParameterAssignment are applied as 'error'.

Remaining difference between 'strict' and 'current' is about code complexity.

We are going to keep them as warning as in many case, they have no functional impact.

New code will how consider those issues as blocker.