mirror of
https://github.com/PostHog/posthog.git
synced 2024-12-01 04:12:23 +01:00
f06e0b4c51
Here I have installed react-hooks eslint rules. See
https://reactjs.org/docs/hooks-rules.html#eslint-plugin for reference.
The specific trigger for this was [this
fix](abdbe54663)
I put in for a bug, which would have been caught by the
`react-hooks/exhaustive-deps` rule.
I've added these both as warnings as there are currently 69 😏
problems:
```
$ yarn eslint
...
✖ 69 problems (6 errors, 63 warnings)
```
At least this will highlight in editors that support it. We can make
these errors in a follow up PR, but this is valuable in itself without
getting into some yak shaving.
88 lines
2.7 KiB
JavaScript
88 lines
2.7 KiB
JavaScript
/* eslint-disable @typescript-eslint/no-var-requires */
|
|
/* global module */
|
|
|
|
module.exports = {
|
|
env: {
|
|
browser: true,
|
|
es6: true,
|
|
'cypress/globals': true,
|
|
},
|
|
settings: {
|
|
react: {
|
|
version: 'detect',
|
|
},
|
|
},
|
|
extends: ['plugin:@typescript-eslint/recommended', 'plugin:react/recommended', 'prettier'],
|
|
globals: {
|
|
Atomics: 'readonly',
|
|
SharedArrayBuffer: 'readonly',
|
|
},
|
|
parser: '@typescript-eslint/parser',
|
|
parserOptions: {
|
|
ecmaFeatures: {
|
|
jsx: true,
|
|
},
|
|
ecmaVersion: 2018,
|
|
sourceType: 'module',
|
|
},
|
|
plugins: ['prettier', 'react', 'react-hooks', 'cypress', '@typescript-eslint'],
|
|
rules: {
|
|
'react/prop-types': [0],
|
|
'react/no-unescaped-entities': [0],
|
|
'react/jsx-no-target-blank': [0],
|
|
'react/self-closing-comp': [
|
|
'error',
|
|
{
|
|
component: true,
|
|
html: true,
|
|
},
|
|
],
|
|
'react-hooks/rules-of-hooks': 'warn',
|
|
'react-hooks/exhaustive-deps': 'warn',
|
|
'no-unused-vars': ['error', { ignoreRestSiblings: true }],
|
|
'@typescript-eslint/explicit-function-return-type': 'off',
|
|
'@typescript-eslint/explicit-module-boundary-types': 'off',
|
|
'@typescript-eslint/no-empty-function': 'off',
|
|
'@typescript-eslint/no-inferrable-types': 'off',
|
|
'@typescript-eslint/ban-ts-comment': 'off',
|
|
'no-shadow': 'error',
|
|
'@typescript-eslint/no-non-null-assertion': 'error',
|
|
curly: 'error',
|
|
},
|
|
overrides: [
|
|
{
|
|
// enable the rule specifically for TypeScript files
|
|
files: ['*Type.ts', '*Type.tsx'],
|
|
rules: {
|
|
'@typescript-eslint/no-explicit-any': ['off'],
|
|
'@typescript-eslint/ban-types': ['off'],
|
|
},
|
|
},
|
|
{
|
|
// enable the rule specifically for TypeScript files
|
|
files: ['*.ts', '*.tsx'],
|
|
rules: {
|
|
'@typescript-eslint/no-explicit-any': ['off'],
|
|
'@typescript-eslint/explicit-function-return-type': [
|
|
'error',
|
|
{
|
|
allowExpressions: true,
|
|
},
|
|
],
|
|
'@typescript-eslint/explicit-module-boundary-types': [
|
|
'error',
|
|
{
|
|
allowArgumentsExplicitlyTypedAsAny: true,
|
|
},
|
|
],
|
|
},
|
|
},
|
|
{
|
|
files: ['*.js'],
|
|
rules: {
|
|
'@typescript-eslint/no-var-requires': 'off',
|
|
},
|
|
},
|
|
],
|
|
}
|