Skip to content

Commit

Permalink
remove unnecessary lint (#20)
Browse files Browse the repository at this point in the history
* move eslintrc

* update lint script

I don't need lints for markup or yaml files

* remove unused files

* add tsconfig.lint.json

* remove lint workflow
`pnpm run lint` is run in ci.yml
  • Loading branch information
tnyo43 authored Mar 20, 2024
1 parent d8fb4e9 commit 832362c
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 78 deletions.
3 changes: 1 addition & 2 deletions .github/linters/.eslintrc.yml → .eslintrc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@ parserOptions:
ecmaVersion: 2023
sourceType: module
project:
- './.github/linters/tsconfig.json'
- './tsconfig.json'
- './tsconfig.lint.json'

plugins:
- jest
Expand Down
18 changes: 0 additions & 18 deletions .github/linters/.markdown-lint.yml

This file was deleted.

10 changes: 0 additions & 10 deletions .github/linters/.yaml-lint.yml

This file was deleted.

9 changes: 0 additions & 9 deletions .github/linters/tsconfig.json

This file was deleted.

38 changes: 0 additions & 38 deletions .github/workflows/linter.yml

This file was deleted.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
"coverage": "make-coverage-badge --output-path ./badges/coverage.svg",
"format:write": "prettier --write **/*.ts",
"format:check": "prettier --check **/*.ts",
"lint": "npx eslint . -c ./.github/linters/.eslintrc.yml",
"lint": "npx eslint . -c .eslintrc.yml",
"package": "ncc build src/index.ts --license licenses.txt",
"package:watch": "npm run package -- --watch",
"test": "jest",
Expand Down
9 changes: 9 additions & 0 deletions tsconfig.lint.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
{
"$schema": "https://json.schemastore.org/tsconfig",
"extends": "./tsconfig.json",
"compilerOptions": {
"noEmit": true
},
"include": ["./__tests__/**/*", "./src/**/*"],
"exclude": ["./dist", "./node_modules", "./coverage", "*.json"]
}

0 comments on commit 832362c

Please sign in to comment.