Skip to content

Commit

Permalink
Fix ts linter (#95)
Browse files Browse the repository at this point in the history
* fix tslinter

* fix tslinter

* bump packages
  • Loading branch information
ktalebian authored Sep 14, 2020
1 parent d344d02 commit 8524379
Show file tree
Hide file tree
Showing 7 changed files with 98 additions and 40 deletions.
6 changes: 3 additions & 3 deletions package-lock.json

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

54 changes: 40 additions & 14 deletions packages/eslint-config-twilio-mocha/package-lock.json

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

14 changes: 7 additions & 7 deletions packages/eslint-config-twilio-react/package-lock.json

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

2 changes: 1 addition & 1 deletion packages/eslint-config-twilio-react/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
"eslint-config-twilio": "^1.29.0",
"eslint-plugin-jsx-a11y": "^6.3.1",
"eslint-plugin-react": "^7.20.6",
"eslint-plugin-react-hooks": "^4.1.0"
"eslint-plugin-react-hooks": "^4.1.2"
},
"peerDependencies": {
"eslint": "^5.0.0 || ^6.0.0 || ^7.0.0"
Expand Down
54 changes: 40 additions & 14 deletions packages/eslint-config-twilio-ts/package-lock.json

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

2 changes: 1 addition & 1 deletion packages/eslint-config-twilio-ts/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,6 @@
},
"peerDependencies": {
"eslint": "^5.0.0 || ^6.0.0 || ^7.0.0",
"typescript": "^3.3.1"
"typescript": "^4.0.2"
}
}
6 changes: 6 additions & 0 deletions packages/eslint-config-twilio-ts/rules/typescript.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
module.exports = {
// https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-shadow.md
'no-shadow': 'off',

// We use Interface instead of PropTypes in TS
'react/prop-types': 'off',

Expand Down Expand Up @@ -46,4 +49,7 @@ module.exports = {
'@typescript-eslint/restrict-plus-operands': 'error',
'@typescript-eslint/typedef': 'off',
'@typescript-eslint/unified-signatures': 'error',

// We don't have to force return type as error
'@typescript-eslint/explicit-module-boundary-types': 'warn',
};

0 comments on commit 8524379

Please sign in to comment.