Fix lint (#16)

* update prettierrc

* remove rules to simplify

* update

.github/linters/.eslintrc.yml

@@ -31,53 +31,4 @@ extends:
   - eslint:recommended
   - plugin:@typescript-eslint/eslint-recommended
   - plugin:@typescript-eslint/recommended
-  - plugin:github/recommended
   - plugin:jest/recommended
-
-rules:
-  {
-    'camelcase': 'off',
-    'eslint-comments/no-use': 'off',
-    'eslint-comments/no-unused-disable': 'off',
-    'i18n-text/no-en': 'off',
-    'import/no-namespace': 'off',
-    'no-console': 'off',
-    'no-unused-vars': 'off',
-    'prettier/prettier': 'error',
-    'semi': 'off',
-    '@typescript-eslint/array-type': 'error',
-    '@typescript-eslint/await-thenable': 'error',
-    '@typescript-eslint/ban-ts-comment': 'error',
-    '@typescript-eslint/consistent-type-assertions': 'error',
-    '@typescript-eslint/explicit-member-accessibility':
-      ['error', { 'accessibility': 'no-public' }],
-    '@typescript-eslint/explicit-function-return-type':
-      ['error', { 'allowExpressions': true }],
-    '@typescript-eslint/func-call-spacing': ['error', 'never'],
-    '@typescript-eslint/no-array-constructor': 'error',
-    '@typescript-eslint/no-empty-interface': 'error',
-    '@typescript-eslint/no-explicit-any': 'error',
-    '@typescript-eslint/no-extraneous-class': 'error',
-    '@typescript-eslint/no-for-in-array': 'error',
-    '@typescript-eslint/no-inferrable-types': 'error',
-    '@typescript-eslint/no-misused-new': 'error',
-    '@typescript-eslint/no-namespace': 'error',
-    '@typescript-eslint/no-non-null-assertion': 'warn',
-    '@typescript-eslint/no-require-imports': 'error',
-    '@typescript-eslint/no-unnecessary-qualifier': 'error',
-    '@typescript-eslint/no-unnecessary-type-assertion': 'error',
-    '@typescript-eslint/no-unused-vars': 'error',
-    '@typescript-eslint/no-useless-constructor': 'error',
-    '@typescript-eslint/no-var-requires': 'error',
-    '@typescript-eslint/prefer-for-of': 'warn',
-    '@typescript-eslint/prefer-function-type': 'warn',
-    '@typescript-eslint/prefer-includes': 'error',
-    '@typescript-eslint/prefer-string-starts-ends-with': 'error',
-    '@typescript-eslint/promise-function-async': 'error',
-    '@typescript-eslint/require-array-sort-compare': 'error',
-    '@typescript-eslint/restrict-plus-operands': 'error',
-    '@typescript-eslint/semi': ['error', 'never'],
-    '@typescript-eslint/space-before-function-paren': 'off',
-    '@typescript-eslint/type-annotation-spacing': 'error',
-    '@typescript-eslint/unbound-method': 'error'
-  }

.prettierrc.json

@@ -1,16 +1,7 @@
 {
-  "printWidth": 80,
   "tabWidth": 2,
-  "useTabs": false,
-  "semi": false,
+  "semi": true,
   "singleQuote": true,
-  "quoteProps": "as-needed",
-  "jsxSingleQuote": false,
-  "trailingComma": "none",
-  "bracketSpacing": true,
-  "bracketSameLine": true,
-  "arrowParens": "avoid",
-  "proseWrap": "always",
-  "htmlWhitespaceSensitivity": "css",
-  "endOfLine": "lf"
+  "trailingComma": "all",
+  "arrowParens": "always"
 }

__tests__/index.test.ts

@@ -2,16 +2,16 @@
  * Unit tests for the action's entrypoint, src/index.ts
  */
 
-import * as main from '../src/main'
+import * as main from '../src/main';
 
 // Mock the action's entrypoint
-const runMock = jest.spyOn(main, 'run').mockImplementation()
+const runMock = jest.spyOn(main, 'run').mockImplementation();
 
 describe('index', () => {
   it('calls run when imported', async () => {
     // eslint-disable-next-line @typescript-eslint/no-require-imports
-    require('../src/index')
+    require('../src/index');
 
-    expect(runMock).toHaveBeenCalled()
-  })
-})
+    expect(runMock).toHaveBeenCalled();
+  });
+});

__tests__/main.test.ts

@@ -1,4 +1,4 @@
-test.todo('implement test')
+test.todo('implement test');
 
 // /**
 //  * Unit tests for the action's main functionality, src/main.ts
@@ -8,84 +8,87 @@ test.todo('implement test')
 //  * variables following the pattern `INPUT_<INPUT_NAME>`.
 //  */
 
-// import * as core from '@actions/core'
-// import * as main from '../src/main'
+// import * as core from '@actions/core';
+// import * as main from '../src/main';
 
 // // 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}/;
 
 // // Mock the GitHub Actions core library
-// let debugMock: jest.SpiedFunction<typeof core.debug>
-// let errorMock: jest.SpiedFunction<typeof core.error>
-// let getInputMock: jest.SpiedFunction<typeof core.getInput>
-// let setFailedMock: jest.SpiedFunction<typeof core.setFailed>
-// let setOutputMock: jest.SpiedFunction<typeof core.setOutput>
+// let debugMock: jest.SpiedFunction<typeof core.debug>;
+// let errorMock: jest.SpiedFunction<typeof core.error>;
+// let getInputMock: jest.SpiedFunction<typeof core.getInput>;
+// let setFailedMock: jest.SpiedFunction<typeof core.setFailed>;
+// let setOutputMock: jest.SpiedFunction<typeof core.setOutput>;
 
 // describe('action', () => {
 //   beforeEach(() => {
-//     jest.clearAllMocks()
+//     jest.clearAllMocks();
 
-//     debugMock = jest.spyOn(core, 'debug').mockImplementation()
-//     errorMock = jest.spyOn(core, 'error').mockImplementation()
-//     getInputMock = jest.spyOn(core, 'getInput').mockImplementation()
-//     setFailedMock = jest.spyOn(core, 'setFailed').mockImplementation()
-//     setOutputMock = jest.spyOn(core, 'setOutput').mockImplementation()
-//   })
+//     debugMock = jest.spyOn(core, 'debug').mockImplementation();
+//     errorMock = jest.spyOn(core, 'error').mockImplementation();
+//     getInputMock = jest.spyOn(core, 'getInput').mockImplementation();
+//     setFailedMock = jest.spyOn(core, 'setFailed').mockImplementation();
+//     setOutputMock = jest.spyOn(core, 'setOutput').mockImplementation();
+//   });
 
 //   it('sets the time output', async () => {
 //     // Set the action's inputs as return values from core.getInput()
-//     getInputMock.mockImplementation(name => {
+//     getInputMock.mockImplementation((name) => {
 //       switch (name) {
 //         case 'milliseconds':
-//           return '500'
+//           return '500';
 //         default:
-//           return ''
+//           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(
+//       1,
+//       'Waiting 500 milliseconds ...',
+//     );
 //     expect(debugMock).toHaveBeenNthCalledWith(
 //       2,
-//       expect.stringMatching(timeRegex)
-//     )
+//       expect.stringMatching(timeRegex),
+//     );
 //     expect(debugMock).toHaveBeenNthCalledWith(
 //       3,
-//       expect.stringMatching(timeRegex)
-//     )
+//       expect.stringMatching(timeRegex),
+//     );
 //     expect(setOutputMock).toHaveBeenNthCalledWith(
 //       1,
 //       'time',
-//       expect.stringMatching(timeRegex)
-//     )
-//     expect(errorMock).not.toHaveBeenCalled()
-//   })
+//       expect.stringMatching(timeRegex),
+//     );
+//     expect(errorMock).not.toHaveBeenCalled();
+//   });
 
 //   it('sets a failed status', async () => {
 //     // Set the action's inputs as return values from core.getInput()
-//     getInputMock.mockImplementation(name => {
+//     getInputMock.mockImplementation((name) => {
 //       switch (name) {
 //         case 'milliseconds':
-//           return 'this is not a number'
+//           return 'this is not a number';
 //         default:
-//           return ''
+//           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'
-//     )
-//     expect(errorMock).not.toHaveBeenCalled()
-//   })
-// })
+//       'milliseconds not a number',
+//     );
+//     expect(errorMock).not.toHaveBeenCalled();
+//   });
+// });

__tests__/wait.test.ts

@@ -2,23 +2,23 @@
  * Unit tests for src/wait.ts
  */
 
-import { wait } from '../src/wait'
+import { wait } from '../src/wait';
 
 describe('wait.ts', () => {
   it('throws an invalid number', async () => {
-    const input = parseInt('foo', 10)
-    expect(isNaN(input)).toBe(true)
+    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()
+    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);
+  });
+});

package.json

@@ -77,7 +77,6 @@
     "@typescript-eslint/parser": "^7.1.0",
     "@vercel/ncc": "^0.38.1",
     "eslint": "^8.56.0",
-    "eslint-plugin-github": "^4.10.1",
     "eslint-plugin-jest": "^27.9.0",
     "eslint-plugin-jsonc": "^2.13.0",
     "eslint-plugin-prettier": "^5.1.3",