Skip to content

Commit

Permalink
chore: update eslint to sort out imports (#1655)
Browse files Browse the repository at this point in the history
* chore: update eslint to sort out imports

* turn warnings into errrors
  • Loading branch information
tatomyr authored Aug 12, 2024
1 parent 47aa04e commit d0fc69c
Show file tree
Hide file tree
Showing 131 changed files with 913 additions and 352 deletions.
15 changes: 13 additions & 2 deletions .eslintrc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ parserOptions:
sourceType: module
plugins:
- '@typescript-eslint'
- import
rules:
'@typescript-eslint/no-unused-vars':
- error
Expand All @@ -20,10 +21,20 @@ rules:
'@typescript-eslint/no-var-requires': off
'@typescript-eslint/no-empty-function': off
'@typescript-eslint/no-inferrable-types': off
'@typescript-eslint/ban-types': warn
'@typescript-eslint/ban-types': error
'@typescript-eslint/no-explicit-any': warn # TODO: make it error!
no-prototype-builtins: off
no-useless-escape: warn
no-useless-escape: error
'@typescript-eslint/consistent-type-imports':
- error
- prefer: type-imports
import/order:
- error
- groups:
- ['builtin', 'external', 'parent', 'sibling', 'index', 'object']
- [type]
'newlines-between': 'always'
import/no-duplicates: error
ignorePatterns:
- '**/__tests__/'
- 'packages/*/lib/'
Expand Down
3 changes: 2 additions & 1 deletion benchmark/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@
"cli-1.15": "npm:@redocly/[email protected]",
"cli-1.16": "npm:@redocly/[email protected]",
"cli-1.17": "npm:@redocly/[email protected]",
"cli-1.18": "npm:@redocly/[email protected]",
"cli-1.18": "npm:@redocly/[email protected]",
"cli-1.19": "npm:@redocly/[email protected]",
"cli-next": "file:../redocly-cli.tgz"
}
}
4 changes: 2 additions & 2 deletions jest.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ module.exports = {
coverageThreshold: {
'packages/core/': {
statements: 80,
branches: 71,
functions: 73,
branches: 72,
functions: 74,
lines: 80,
},
'packages/cli/': {
Expand Down
Loading

1 comment on commit d0fc69c

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 77.5% 4664/6018
🟡 Branches 65.99% 1938/2937
🟡 Functions 71.38% 763/1069
🟡 Lines 77.71% 4401/5663

Test suite run success

759 tests passing in 107 suites.

Report generated by 🧪jest coverage report action from d0fc69c

Please sign in to comment.