skatteetaten/checkstyle-config

Checkstyle Configuration for Skatteetaten

Checkstyle Config

Contains the checkstyle configuration used for all Java code at Skatteetaten. It is mandatory to include one of the two profiles in your build for internal code at Skatteetaten.

checkstyle-config's Issues

Øke linjelengde til 180 tegn

Jeg vil gjerne forslå at vi øker antall tegn per linje. I praksis blir denne regelen oftest ignorert og takket 'broken window theorem' fører det ofte at man også ignorerer andre regler.

180 tegn kan vises på nesten alle skjerm og innimellom er lange linjer mer lesbar enn mange.

Enable ignoreOverriddenMethods on ParameterNumber

When overriding a method that has more than the permitted number of parameters, the ruleset currently complains at the site of the overriding method. This makes little sense, as there's unlikely that there is anything the implementor of the overriding method can do, except to add a @Suppress("ParameterNumber") - it's the overridden method that is the problem.
Therefore we should enable the ignoreOverriddenMethods flag on the check, so that users don't need to add suppressions themselves.

Sjekk for LineLength bør inkludere fileExtensions

I dag blir også properties filer inkludert og sjekket for linje lengde. Det er uheldig. Vurdere om fileExtensions bør inkluderes (java++?)