Skip to content
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

[web] Add eslint-plugin-simple-import-sort #49368

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kiosion
Copy link
Contributor

@kiosion kiosion commented Nov 22, 2024

  • Add config for eslint-plugin-simple-import-sort: Serves same purpose as current import/order rule, but has better support for autofix, prevents newlines accumulating (currently, running import auto fix can lead to extra newlines being left in place), and has 0 dependencies
    • New import sorting order is identical to the prior config, but now sorts within import groups alphabetically, and within destructured imports alphabetically.
  • Stub out some useful @typescript-eslint rules we should enable eventually

* Add config for `eslint-plugin-simple-import-sort`: Serves same purpose
  as current `import/order` rule, but has better support for autofix,
  prevents newlines accumulating, and has 0 dependencies
* Stub out some useful `@typescript-eslint` rules we should enable
  eventually
@kiosion kiosion added do-not-merge no-changelog Indicates that a PR does not require a changelog entry labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge no-changelog Indicates that a PR does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant