Chore/migrate eslint

.github/linters/.eslintrc.yml

@@ -26,12 +26,12 @@ parserOptions:
 plugins:
   - jest
   - '@typescript-eslint'
+  - prettier
 
 extends:
   - eslint:recommended
   - plugin:@typescript-eslint/eslint-recommended
   - plugin:@typescript-eslint/recommended
-  - plugin:github/recommended
   - plugin:jest/recommended
 
 rules: {
@@ -46,7 +46,7 @@ rules: {
     'no-console': 'off',
     'no-unused-vars': 'off',
     'prettier/prettier': 'error',
-    'semi': 'off',
+    'semi': ['error', 'never'],
     '@typescript-eslint/array-type': 'error',
     '@typescript-eslint/await-thenable': 'error',
     '@typescript-eslint/ban-ts-comment': 'error',
@@ -55,7 +55,7 @@ rules: {
       ['error', { 'accessibility': 'no-public' }],
     '@typescript-eslint/explicit-function-return-type':
       ['error', { 'allowExpressions': true }],
-    '@typescript-eslint/func-call-spacing': ['error', 'never'],
+    'func-call-spacing': ['error', 'never'],
     '@typescript-eslint/no-array-constructor': 'error',
     '@typescript-eslint/no-empty-interface': 'error',
     '@typescript-eslint/no-explicit-any': 'error',
@@ -68,7 +68,18 @@ rules: {
     '@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-unused-vars':
+      [
+        'error',
+        {
+          'caughtErrors': 'none',
+          'argsIgnorePattern': '^_',
+          'varsIgnorePattern': '^_',
+          'ignoreRestSiblings': true
+        }
+      ],
+    '@typescript-eslint/no-unused-expressions':
+      ['error', { 'allowShortCircuit': true }],
     '@typescript-eslint/no-useless-constructor': 'error',
     '@typescript-eslint/no-var-requires': 'error',
     '@typescript-eslint/prefer-for-of': 'warn',
@@ -78,8 +89,8 @@ rules: {
     '@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/semi': 'off',
     '@typescript-eslint/space-before-function-paren': 'off',
-    '@typescript-eslint/type-annotation-spacing': 'error',
+    '@typescript-eslint/type-annotation-spacing': 'off',
     '@typescript-eslint/unbound-method': 'error'
   }

.github/linters/.gitleaks.toml

@@ -0,0 +1,5 @@
+[allowlist]
+  description = "Allowlisted paths"
+  paths = [
+    '''dist/'''
+  ]

README.md

@@ -253,5 +253,5 @@ The scripts and documentation in this project are released under the
 
 ## Contributing
 
-Please read [CONTRIBUTING.md](./CONTRIBUTING.md) for details on our code of
+Please read [CONTRIBUTION.md](./CONTRIBUTION.md) for details on our code of
 conduct, and the process for submitting pull requests to us.

__mocks__/@actions/core.cjs

@@ -0,0 +1,4 @@
+exports.getInput = () => ''
+exports.setOutput = () => { }
+exports.setFailed = () => { }
+exports.debug = () => { }

__tests__/main.test.ts

@@ -114,4 +114,14 @@ describe('run function', () => {
     // Expectations
     expect(core.debug).toHaveBeenCalledTimes(14) // Adjust the count based on your actual number of debug logs
   })
+
+  it('should not call setFailed for non-Error throws', async () => {
+    jest.spyOn(core, 'getInput').mockImplementationOnce(() => {
+      throw 'non-error throw'
+    })
+
+    await run()
+
+    expect(core.setFailed).not.toHaveBeenCalled()
+  })
 })