added lint to CI workflow

Author: mtrezza

.editorconfig

@@ -2,7 +2,7 @@ root = true
 
 [*]
 indent_style = space
-indent_size = 4
+indent_size = 2
 end_of_line = lf
 charset = utf-8
 trim_trailing_whitespace = true

.github/workflows/ci.yml

@@ -23,8 +23,8 @@ jobs:
           key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |
             ${{ runner.os }}-
-      - name: Install dependencies
-        run: npm ci
+      - run: npm ci
+      - run: npm run lint
       - run: npm test
         env:
           CI: true

.github/workflows/release.yml

@@ -19,7 +19,6 @@ jobs:
           restore-keys: |
               ${{ runner.os }}-node-
       - run: npm ci
-      - run: npm run release
       - run: npm publish
         env:
           NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}

package.json

@@ -8,10 +8,11 @@
     "build": "babel src --out-dir lib --source-maps",
     "lint:src": "eslint 'src/**/*.js'",
     "lint:test": "eslint 'spec/**/*.js'",
+    "lint:fix": "eslint 'spec/**/*.js' --fix && eslint 'spec/**/*.js' --fix",
     "lint": "npm run lint:src && npm run lint:test",
     "test": "nyc --reporter=lcov jasmine",
-    "release": "npm run build && npm test",
     "posttest": "nyc report --reporter=json && codecov -f coverage/*.json",
+    "prepare": "npm run build && npm test",
     "demo": "node ./demo"
   },
   "repository": {

spec/.eslintrc.json

@@ -0,0 +1,11 @@
+{
+    "env": {
+        "jasmine": true
+    },
+    "globals": {
+    },
+    "rules": {
+      "no-console": [0],
+      "no-var": "error"
+    }
+}

spec/ApiMailAdapter.spec.js

@@ -4,21 +4,21 @@ const MailAdapter = require('../src/MailAdapter');
 const { Parse } = require('./helper');
 
 describe('MailAdapter', () => {
-    const adapter = new MailAdapter()
-    const user = new Parse.User();
+  const adapter = new MailAdapter()
+  const user = new Parse.User();
 
-    it('should have a method called sendMail', () => {
-        expect(typeof adapter.sendMail).toBe('function');
-        expect(adapter.sendMail()).toBeUndefined();
-    });
+  it('should have a method called sendMail', () => {
+    expect(typeof adapter.sendMail).toBe('function');
+    expect(adapter.sendMail()).toBeUndefined();
+  });
 
-    it('should have a method called sendVerificationEmail', () => {
-        expect(typeof adapter.sendVerificationEmail).toBe('function');
-        expect(adapter.sendVerificationEmail({ link: 'link', appName: 'appName', user: user })).toBeUndefined();
-    });
+  it('should have a method called sendVerificationEmail', () => {
+    expect(typeof adapter.sendVerificationEmail).toBe('function');
+    expect(adapter.sendVerificationEmail({ link: 'link', appName: 'appName', user: user })).toBeUndefined();
+  });
 
-    it('should have a method called sendPasswordResetEmail', () => {
-        expect(typeof adapter.sendPasswordResetEmail).toBe('function');
-        expect(adapter.sendPasswordResetEmail({ link: 'link', appName: 'appName', user: user })).toBeUndefined();
-    });
-});
+  it('should have a method called sendPasswordResetEmail', () => {
+    expect(typeof adapter.sendPasswordResetEmail).toBe('function');
+    expect(adapter.sendPasswordResetEmail({ link: 'link', appName: 'appName', user: user })).toBeUndefined();
+  });
+});

spec/MailAdapter.spec.js

@@ -2,18 +2,18 @@
 
 // Simluate Parse User class
 const Parse = {
-    User: class User {
-        get(key) {
-            switch (key) {
-                case 'username':
-                    return 'ExampleUsername'
-                case 'email':
-                    return '[email protected]'
-            }
-        }
+  User: class User {
+    get(key) {
+      switch (key) {
+        case 'username':
+          return 'ExampleUsername'
+        case 'email':
+          return '[email protected]'
+      }
     }
+  }
 };
 
 module.exports = {
-    Parse
-};
+  Parse
+};