MaxLineLength should allow excluding import and package statements

Hey,

Another issue i found is MaxLineLength, which is not working, i changed it to 15 just to see the results, and build has succeed with no warns or errors.

detekt.yml part:

style:
  active: true
  WildcardImport:
    active: false
  MaxLineLength:
    active: true
    maxLineLength: 15
  NamingConventionViolation:
    active: true
    variablePattern: '^(_)?[a-z$][a-zA-Z$0-9]*$'
    constantPattern: '^([A-Z_]*|serialVersionUID)$'
    methodPattern: "^[`]?[a-z$'][a-zA-Z$0-9]*[`]?$"
    classPattern: '[A-Z$][a-zA-Z$]*'
    enumEntryPattern: '^[A-Z$][a-zA-Z_$]*$'

[Mauin]

@arturbosch Will do that on the weekend 👍

[arturbosch]

Puh, I actually do not like that you can exclude classes, methods or properties etc. If you are on the level where you want to exclude whole classes etc then just turn off the rule … or use @Suppress or add this findings to your white- or blacklist, create a baseline. So many options.

For package decl and import stmt’s I see the need if your company has an extravagant name 😄.

@Mauin go for excludeImports and excludePackages, both false on default. All agree 😄 ?