Review eslint rules

[apoclyps]

The following rules have been applied to the project to allow a more gradual migration to a set standard; review the existing rules and make changes were necessary. We should avoid disabling rules to avoid making changes to the existing codebase.

    "import/no-extraneous-dependencies": "off",
    "import/no-unresolved": "off",
    "import/first": "off",
    "import/no-undef": "off",
    "import/extensions": "off",
    "class-methods-use-this": "off",
    "react/prefer-stateless-function": "off",
    "consistent-return": "off",
    "jsx-a11y/click-events-have-key-events": "off",
    "import/no-duplicates": "off",
    "no-console": "off",
    "jsx-a11y/no-static-element-interactions": "off",
    "no-use-before-define": "off",
    "no-param-reassign": "off",
    "react/jsx-no-bind": "off",
    "react/no-string-refs": "off",
    "babel/no-invalid-this": "off",
    "no-undef": "off",
    "react/jsx-no-duplicate-props": "off",
    "no-unused-vars": "off",
    "no-debugger": "off"

[alistairjcbrown]

We can probably close this off with the simplified set we're now using from prettier / airbnb - #205