Update linter

Author: crazywhalecc

.github/linters/.eslintrc.yml

@@ -46,5 +46,5 @@ rules:
     'no-console': 'off',
     'no-unused-vars': 'off',
     'prettier/prettier': 'error',
-    'semi': 'off'
+    'semi': 'off',
   }

.prettierrc.json

@@ -1,16 +1,24 @@
 {
   "printWidth": 80,
-  "tabWidth": 4,
+  "tabWidth": 2,
   "useTabs": false,
   "semi": true,
   "singleQuote": true,
   "quoteProps": "as-needed",
   "jsxSingleQuote": false,
-  "trailingComma": "all",
+  "trailingComma": "es5",
   "bracketSpacing": true,
   "bracketSameLine": true,
   "arrowParens": "avoid",
   "proseWrap": "always",
   "htmlWhitespaceSensitivity": "css",
-  "endOfLine": "lf"
+  "endOfLine": "lf",
+  "overrides": [
+    {
+      "files": "*.js",
+      "options": {
+        "tabWidth": 4
+      }
+    }
+  ]
 }

README.md

@@ -88,14 +88,14 @@ There are a few things to keep in mind when writing your action code:
   In `main.js`, you will see that the action is run in an `async` function.
 
   ```javascript
-  const core = require('@actions/core')
+  const core = require('@actions/core');
   //...
 
   async function run() {
     try {
       //...
     } catch (error) {
-      core.setFailed(error.message)
+      core.setFailed(error.message);
     }
   }
   ```

__tests__/index.test.js

@@ -2,17 +2,17 @@
  * Unit tests for the action's entrypoint, src/index.js
  */
 
-const { run } = require('../src/main')
+const { run } = require('../src/main');
 
 // Mock the action's entrypoint
 jest.mock('../src/main', () => ({
-  run: jest.fn()
-}))
+    run: jest.fn(),
+}));
 
 describe('index', () => {
-  it('calls run when imported', async () => {
-    require('../src/index')
+    it('calls run when imported', async () => {
+        require('../src/index');
 
-    expect(run).toHaveBeenCalled()
-  })
-})
+        expect(run).toHaveBeenCalled();
+    });
+});

__tests__/main.test.js

@@ -1,96 +1,101 @@
 /**
  * Unit tests for the action's main functionality, src/main.js
  */
-const core = require('@actions/core')
-const main = require('../src/main')
+const core = require('@actions/core');
+const main = require('../src/main');
 
 // Mock the GitHub Actions core library
-const debugMock = jest.spyOn(core, 'debug').mockImplementation()
-const getInputMock = jest.spyOn(core, 'getInput').mockImplementation()
-const setFailedMock = jest.spyOn(core, 'setFailed').mockImplementation()
-const setOutputMock = jest.spyOn(core, 'setOutput').mockImplementation()
+const debugMock = jest.spyOn(core, 'debug').mockImplementation();
+const getInputMock = jest.spyOn(core, 'getInput').mockImplementation();
+const setFailedMock = jest.spyOn(core, 'setFailed').mockImplementation();
+const setOutputMock = jest.spyOn(core, 'setOutput').mockImplementation();
 
 // Mock the action's main function
-const runMock = jest.spyOn(main, 'run')
+const runMock = jest.spyOn(main, 'run');
 
 // Other utilities
-const timeRegex = /^\d{2}:\d{2}:\d{2}/
+const timeRegex = /^\d{2}:\d{2}:\d{2}/;
 
 describe('action', () => {
-  beforeEach(() => {
-    jest.clearAllMocks()
-  })
+    beforeEach(() => {
+        jest.clearAllMocks();
+    });
 
-  it('sets the time output', async () => {
-    // Set the action's inputs as return values from core.getInput()
-    getInputMock.mockImplementation(name => {
-      switch (name) {
-        case 'milliseconds':
-          return '500'
-        default:
-          return ''
-      }
-    })
+    it('sets the time output', async () => {
+        // Set the action's inputs as return values from core.getInput()
+        getInputMock.mockImplementation(name => {
+            switch (name) {
+                case 'milliseconds':
+                    return '500';
+                default:
+                    return '';
+            }
+        });
 
-    await main.run()
-    expect(runMock).toHaveReturned()
+        await main.run();
+        expect(runMock).toHaveReturned();
 
-    // Verify that all of the core library functions were called correctly
-    expect(debugMock).toHaveBeenNthCalledWith(1, 'Waiting 500 milliseconds ...')
-    expect(debugMock).toHaveBeenNthCalledWith(
-      2,
-      expect.stringMatching(timeRegex)
-    )
-    expect(debugMock).toHaveBeenNthCalledWith(
-      3,
-      expect.stringMatching(timeRegex)
-    )
-    expect(setOutputMock).toHaveBeenNthCalledWith(
-      1,
-      'time',
-      expect.stringMatching(timeRegex)
-    )
-  })
+        // Verify that all of the core library functions were called correctly
+        expect(debugMock).toHaveBeenNthCalledWith(
+            1,
+            'Waiting 500 milliseconds ...'
+        );
+        expect(debugMock).toHaveBeenNthCalledWith(
+            2,
+            expect.stringMatching(timeRegex)
+        );
+        expect(debugMock).toHaveBeenNthCalledWith(
+            3,
+            expect.stringMatching(timeRegex)
+        );
+        expect(setOutputMock).toHaveBeenNthCalledWith(
+            1,
+            'time',
+            expect.stringMatching(timeRegex)
+        );
+    });
 
-  it('sets a failed status', async () => {
-    // Set the action's inputs as return values from core.getInput()
-    getInputMock.mockImplementation(name => {
-      switch (name) {
-        case 'milliseconds':
-          return 'this is not a number'
-        default:
-          return ''
-      }
-    })
+    it('sets a failed status', async () => {
+        // Set the action's inputs as return values from core.getInput()
+        getInputMock.mockImplementation(name => {
+            switch (name) {
+                case 'milliseconds':
+                    return 'this is not a number';
+                default:
+                    return '';
+            }
+        });
 
-    await main.run()
-    expect(runMock).toHaveReturned()
+        await main.run();
+        expect(runMock).toHaveReturned();
 
-    // Verify that all of the core library functions were called correctly
-    expect(setFailedMock).toHaveBeenNthCalledWith(
-      1,
-      'milliseconds not a number'
-    )
-  })
+        // Verify that all of the core library functions were called correctly
+        expect(setFailedMock).toHaveBeenNthCalledWith(
+            1,
+            'milliseconds not a number'
+        );
+    });
 
-  it('fails if no input is provided', async () => {
-    // Set the action's inputs as return values from core.getInput()
-    getInputMock.mockImplementation(name => {
-      switch (name) {
-        case 'milliseconds':
-          throw new Error('Input required and not supplied: milliseconds')
-        default:
-          return ''
-      }
-    })
+    it('fails if no input is provided', async () => {
+        // Set the action's inputs as return values from core.getInput()
+        getInputMock.mockImplementation(name => {
+            switch (name) {
+                case 'milliseconds':
+                    throw new Error(
+                        'Input required and not supplied: milliseconds'
+                    );
+                default:
+                    return '';
+            }
+        });
 
-    await main.run()
-    expect(runMock).toHaveReturned()
+        await main.run();
+        expect(runMock).toHaveReturned();
 
-    // Verify that all of the core library functions were called correctly
-    expect(setFailedMock).toHaveBeenNthCalledWith(
-      1,
-      'Input required and not supplied: milliseconds'
-    )
-  })
-})
+        // Verify that all of the core library functions were called correctly
+        expect(setFailedMock).toHaveBeenNthCalledWith(
+            1,
+            'Input required and not supplied: milliseconds'
+        );
+    });
+});

__tests__/wait.test.js

@@ -1,24 +1,24 @@
 /**
  * Unit tests for src/wait.js
  */
-const { wait } = require('../src/wait')
-const { expect } = require('@jest/globals')
+const { wait } = require('../src/wait');
+const { expect } = require('@jest/globals');
 
 describe('wait.js', () => {
-  it('throws an invalid number', async () => {
-    const input = parseInt('foo', 10)
-    expect(isNaN(input)).toBe(true)
+    it('throws an invalid number', async () => {
+        const input = parseInt('foo', 10);
+        expect(isNaN(input)).toBe(true);
 
-    await expect(wait(input)).rejects.toThrow('milliseconds not a number')
-  })
+        await expect(wait(input)).rejects.toThrow('milliseconds not a number');
+    });
 
-  it('waits with a valid number', async () => {
-    const start = new Date()
-    await wait(500)
-    const end = new Date()
+    it('waits with a valid number', async () => {
+        const start = new Date();
+        await wait(500);
+        const end = new Date();
 
-    const delta = Math.abs(end.getTime() - start.getTime())
+        const delta = Math.abs(end.getTime() - start.getTime());
 
-    expect(delta).toBeGreaterThan(450)
-  })
-})
+        expect(delta).toBeGreaterThan(450);
+    });
+});