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/ewavr.js b/lib/compilers/ewavr.js
index 025e0141b..93ed9b1bb 100644
--- a/lib/compilers/ewavr.js
+++ b/lib/compilers/ewavr.js
@@ -22,11 +22,11 @@
 // ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 // POSSIBILITY OF SUCH DAMAGE.
 
-"use strict";
+'use strict';
 
 const BaseCompiler = require('../base-compiler'),
     temp = require('temp'),
-    AsmEWAVRParser = require('../asm-parser-ewavr.js');
+    AsmEWAVRParser = require('../asm-parser-ewavr');
 
 class EWAVRCompiler extends BaseCompiler {
     constructor(info, env) {
@@ -54,7 +54,7 @@ class EWAVRCompiler extends BaseCompiler {
         
         return [
             '-lB', this.filename(outputFilename),
-            '-o', this.filename(outputFilename + '.obj')
+            '-o', this.filename(outputFilename + '.obj'),
         ];
     }
 }