Skip to content

Commit

Permalink
Merge pull request #171 from reviewdog/fix-eslint-warnings-2021-11-09
Browse files Browse the repository at this point in the history
Fix eslint warnings 2021 11 09
  • Loading branch information
shogo82148 committed Nov 9, 2021
2 parents c9d2bba + ca21986 commit ea4409b
Show file tree
Hide file tree
Showing 11 changed files with 6,841 additions and 199 deletions.
7 changes: 2 additions & 5 deletions .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"project": "./tsconfig.json"
},
"rules": {
"i18n-text/no-en": "off",
"eslint-comments/no-use": "off",
"import/no-namespace": "off",
"no-unused-vars": "off",
Expand Down Expand Up @@ -45,6 +46,7 @@
"@typescript-eslint/no-extraneous-class": "error",
"@typescript-eslint/no-for-in-array": "error",
"@typescript-eslint/no-inferrable-types": "error",
"@typescript-eslint/no-floating-promises": "error",
"@typescript-eslint/no-misused-new": "error",
"@typescript-eslint/no-namespace": "error",
"@typescript-eslint/no-non-null-assertion": "warn",
Expand All @@ -59,11 +61,6 @@
"@typescript-eslint/promise-function-async": "error",
"@typescript-eslint/require-array-sort-compare": "error",
"@typescript-eslint/restrict-plus-operands": "error",
"semi": "off",
"@typescript-eslint/semi": [
"error",
"always"
],
"@typescript-eslint/type-annotation-spacing": "error",
"@typescript-eslint/unbound-method": "error"
},
Expand Down
16 changes: 16 additions & 0 deletions .github/workflows/reviewdog.yml
Original file line number Diff line number Diff line change
Expand Up @@ -163,3 +163,19 @@ jobs:
steps:
- uses: actions/checkout@v2
- uses: reviewdog/action-actionlint@v1

eslint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2

- uses: actions/setup-node@v2
with:
node-version: "12"
cache: 'npm'
- run: npm ci
- uses: reviewdog/action-eslint@v1
with:
reporter: github-pr-review
level: warning
eslint_flags: "src/**/*.ts"
4 changes: 2 additions & 2 deletions __test__/installer.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,11 @@ describe('installer', () => {
const dir = await fs.mkdtemp(path.join(tempDir, 'golangci-'));
const golangci = await installer.installGolangciLint('v1.41.1', dir);
await exec.exec(golangci, ['--version']);
}, 10000);
}, 20000);

it('installs the latest version of golangci-lint', async () => {
const dir = await fs.mkdtemp(path.join(tempDir, 'golangci-'));
const golangci = await installer.installGolangciLint('latest', dir);
await exec.exec(golangci, ['--version']);
}, 10000);
}, 20000);
});
100 changes: 47 additions & 53 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

0 comments on commit ea4409b

Please sign in to comment.