-
Notifications
You must be signed in to change notification settings - Fork 392
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[chore] Fix eslint so it uses a common config for packages #1886
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sanity-io/test-studio/oi9594739 |
1d7e3c1
to
6f68507
Compare
6f68507
to
964e3c8
Compare
964e3c8
to
878742d
Compare
878742d
to
8db0509
Compare
8db0509
to
18187f9
Compare
18187f9
to
c3f5555
Compare
c3f5555
to
c0cf4eb
Compare
c0cf4eb
to
7576cdc
Compare
Update: did some changes because the |
7576cdc
to
999912d
Compare
999912d
to
29e7736
Compare
cc56dcd
to
ca1f10e
Compare
… all) - Use a single configuration for all source files (typescript/javascript) - Use the @typescript/eslint-parser for everything - Using sanity-eslint to typescript config too - Disable the '@typescript/no-use-before-define' rule - Disable the 'react/no-multi-comp' rule - Disable the 'react/require-optimization' rule - Disable the 'import/prefer-default-export' rule - Disabling 'import/no-commonjs' in favor of '@typescript-eslint/no-var-requires' - Allow commonjs in certain packages by disabling '@typescript-eslint/no-var-requires'
* [chore] Fix eslint so it uses a common config for packages * [chore] Add JS extensions to lint command * [chore] Move eslint extends config in order to apply correct order of rules * [chore] Postpone introduction of eslint-plugin-react-hooks * [chore] Simplify proposed eslint setup a bit (one config to rule them all) - Use a single configuration for all source files (typescript/javascript) - Use the @typescript/eslint-parser for everything - Using sanity-eslint to typescript config too - Disable the '@typescript/no-use-before-define' rule - Disable the 'react/no-multi-comp' rule - Disable the 'react/require-optimization' rule - Disable the 'import/prefer-default-export' rule - Disabling 'import/no-commonjs' in favor of '@typescript-eslint/no-var-requires' - Allow commonjs in certain packages by disabling '@typescript-eslint/no-var-requires' Co-authored-by: Bjørge Næss <bjoerge@gmail.com>
* [chore] Fix eslint so it uses a common config for packages * [chore] Add JS extensions to lint command * [chore] Move eslint extends config in order to apply correct order of rules * [chore] Postpone introduction of eslint-plugin-react-hooks * [chore] Simplify proposed eslint setup a bit (one config to rule them all) - Use a single configuration for all source files (typescript/javascript) - Use the @typescript/eslint-parser for everything - Using sanity-eslint to typescript config too - Disable the '@typescript/no-use-before-define' rule - Disable the 'react/no-multi-comp' rule - Disable the 'react/require-optimization' rule - Disable the 'import/prefer-default-export' rule - Disabling 'import/no-commonjs' in favor of '@typescript-eslint/no-var-requires' - Allow commonjs in certain packages by disabling '@typescript-eslint/no-var-requires' Co-authored-by: Bjørge Næss <bjoerge@gmail.com>
* [chore] Fix eslint so it uses a common config for packages * [chore] Add JS extensions to lint command * [chore] Move eslint extends config in order to apply correct order of rules * [chore] Postpone introduction of eslint-plugin-react-hooks * [chore] Simplify proposed eslint setup a bit (one config to rule them all) - Use a single configuration for all source files (typescript/javascript) - Use the @typescript/eslint-parser for everything - Using sanity-eslint to typescript config too - Disable the '@typescript/no-use-before-define' rule - Disable the 'react/no-multi-comp' rule - Disable the 'react/require-optimization' rule - Disable the 'import/prefer-default-export' rule - Disabling 'import/no-commonjs' in favor of '@typescript-eslint/no-var-requires' - Allow commonjs in certain packages by disabling '@typescript-eslint/no-var-requires' Co-authored-by: Bjørge Næss <bjoerge@gmail.com>
.eslintrc.js
for all packages, no matter if they include JS or TS or both (no need to add a.eslintrc
in the packages that extends the root config).eslint-plugin-react-hooks
.