Build: stop failing the build on eslint errors (#1594)

`eslint` provides helpful warnings that prevent bugs in our software.
However by running it as a precondition for the build we introduce
artificial obstacles to quick testing and iteration. For instance, we
will fail to build the app if we have an unused variable even though
there's no reason it would break the build.

In this change we're removing the precondition for passing `eslint`
before building the app. This should allow us to more rapidly made small
debugging/testing adjustments while working on fixing or altering the
app.

`eslint` still belongs in the process and we might follow-up on this PR
by introducing it as a `git` hook on `precommit`, a more appropriate
place for the safety check. For now though I am foregoing that work in
order to remove what has been a persistent frustration while working on
the app.

package.json

@@ -60,7 +60,6 @@
     "enzyme-adapter-react-16": "1.14.0",
     "eslint": "6.1.0",
     "eslint-config-prettier": "6.3.0",
-    "eslint-loader": "2.2.1",
     "eslint-plugin-jest": "22.15.0",
     "eslint-plugin-prettier": "3.1.1",
     "eslint-plugin-react": "7.14.3",

webpack.config.js

@@ -27,21 +27,6 @@ module.exports = () => {
     },
     module: {
       rules: [
-        {
-          test: /lib\/.+\.jsx?$/,
-          exclude: /node_modules|lib\/simperium/,
-          enforce: 'pre',
-          use: [
-            {
-              loader: 'eslint-loader',
-              options: {
-                cache: true,
-                configFile: '.eslintrc.json',
-                quiet: true,
-              },
-            },
-          ],
-        },
         {
           test: /\.jsx?$/,
           exclude: /node_modules\/core-js/,