WIP: Cleanup

[lue-bird]

I will add elm/random simplifications next, but first a bit of cleanup. If you don't like certain changes, let's discuss and use one style consistently.

• upgrade elm-review to make it run locally
• de-duplicate Node access calls and prefer desctructuring
• use keepOnlyFix and parenthesizeFix instead where possible
• replace noopFix with more explicit alternatives (toIndetityFix or keepOnlyFix)
• move helpers that were sprinkled between check functions to the right sections (e.g. sequence into -- LIST HELPERS)
• use firstThatReportsError to avoid deep _ -> nesting and decouple sub-rules
• introduce neighboringMap, withBeforeMap
• minor documentation fixes, like adding List.partition (always False)
• factor out sub-rules like fullyAppliedPrefixOperatorChecks
• merge structurally similar cases in expressionVisitorHelp
• qualify more stuff
• prefer uncons :: pattern to list elements [ , ] pattern
• prefer variable pattern instead of record pattern

[lue-bird]

For patterns like

case ModuleNameLookupTable.moduleNameAt lookupTable range of
    Just [ "Basics" ] ->

Do you prefer leaving them as is or changing them to an uncons pattern

case ModuleNameLookupTable.moduleNameAt lookupTable range of
    Just ("Basics" :: []) ->

[jfmengels]

I tend to prefer [ "Basics" ] -> (even with multiple elements [ "A", "B", "C" ] ->) when I know the exact number. So yes, leave them as is