-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
eslint: Clean up deprecated rules (#40459)
There are a lot of rules in our config that are normally disabled by eslint-plugin-prettier/recommended, but we've been turning them back on. Many of them are now deprecated in eslint too (in favor of `@stylistic/eslint-plugin-js`). Let's disable them and let prettier handle it. - comma-spacing - computed-property-spacing - curly - func-call-spacing - jsx-quotes - key-spacing - keyword-spacing - no-extra-semi - no-multiple-empty-lines - no-multi-spaces - no-spaced-func - no-trailing-spaces - object-curly-spacing - operator-linebreak - padded-blocks - react/jsx-curly-spacing - react/jsx-tag-spacing - semi - semi-spacing - space-before-blocks - space-infix-ops - space-in-parens - space-unary-ops A few more we had been disabling even though prettier already disabled them. - comma-dangle - space-before-function-paren And then there are a few deprecated in favor of `eslint-plugin-n` instead. - no-process-exit → n/no-process-exit - no-restricted-imports → n/no-restricted-import - no-restricted-modules → n/no-restricted-require And since we're now bringing in `eslint-plugin-n`, may as well enable a few additional rules from that plugin too. - n/no-deprecated-api - n/no-exports-assign - n/process-exit-as-throw Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/12184535037 Upstream-Ref: Automattic/jetpack@d6d5f2b
- Loading branch information
Showing
2 changed files
with
56 additions
and
56 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.