Add new eslint rules (#2121)

The largest changes here are:
- enforcing single quotes for strings
- enforcing trailing commas where possible

In addition to those we have enabled several eslint plugins:
- plugin:requirejs/recommended, to enforce some conventions in require statements
- plugin:node/recommended, to enforce correct usage of various node.js APIs
- plugin:unicorn/recommended, which contains a pretty mixed bag of useful rules

This PR attempts to not change code behavior when possible.  In cases where fixing
existing code would change semantics, a linting exclusion has been placed in the 
code base to silence the error.  You can find these by searching for `eslint-disable-next-line`.

Co-authored-by: Austin Morton <austinpmorton@gmail.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/lib/compilers/fake-for-test.js b/lib/compilers/fake-for-test.js
index c5d456fbf..ff0c43a7d 100644
--- a/lib/compilers/fake-for-test.js
+++ b/lib/compilers/fake-for-test.js
@@ -29,7 +29,7 @@ class FakeCompiler {
         this.compiler = {
             id: info.id || 'fake-for-test',
             lang: info.lang || 'fake-lang',
-            options: info.options || ''
+            options: info.options || '',
         };
         this.info = info;
     }
@@ -52,8 +52,8 @@ class FakeCompiler {
                 source: source,
                 options: options,
                 backendOptions: backendOptions,
-                filters: filters
-            }
+                filters: filters,
+            },
         }));
     }