Code quality

[afgallo]

Our .eslintrc file disables some rules I'd like to keep:

    "comma-dangle": [0],
    "func-names": [0],
    "vars-on-top": [0],
    "max-len": [0],
    "no-param-reassign": [0],
    "no-shadow": [0],
    "object-shorthand": [0],
    "no-underscore-dangle": [0],
    "newline-per-chained-call": [0],
    "new-cap": [0],
    "no-else-return": [0],
    "consistent-return": [0],
    "prefer-rest-params": [0],
    "object-curly-newline": [ "error", { "multiline": true, "consistent": true } ],
    "prefer-destructuring": [0],
    "no-multi-assign": [0],
    "prefer-spread": [0]

Below is a list of tasks to track progress against the above items:

• comma-dangle
• func-names
• vars-on-top
• max-len
• no-param-reassign
• no-shadow
• object-shorthand
• no-underscore-dangle
• newline-per-chained-call
• new-cap
• no-else-return
• consistent-return
• prefer-rest-params
• object-curly-newline
• prefer-destructuring
• no-multi-assign
• prefer-spread