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, 1 insertions, 1 deletions

diff --git a/lib/compilers/ellcc.js b/lib/compilers/ellcc.js
index 2d025e253..7d74a1eca 100644
--- a/lib/compilers/ellcc.js
+++ b/lib/compilers/ellcc.js
@@ -26,7 +26,7 @@ const
 
 class EllccCompiler extends ClangCompiler {
     getSharedLibraryPathsAsArguments() {
-        const pathFlag = this.compiler.rpathFlag || "-Wl,-rpath,";
+        const pathFlag = this.compiler.rpathFlag || '-Wl,-rpath,';
 
         return this.compiler.ldPath.map(path => pathFlag + path);
     }