-
Notifications
You must be signed in to change notification settings - Fork 3
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
Change the base config's import/order
rule so the unknown
import type now fills the last import group slot
#14
Conversation
…type now fills the last import group slot
78c404f
to
0d20e44
Compare
import/order
rule so the unknown
import type is lastimport/order
rule so the unknown
import type now fills the last import group slot
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some notes for context:
@@ -39,9 +39,10 @@ | |||
{ | |||
"groups": [ | |||
["builtin", "external"], | |||
["internal", "unknown", "parent", "sibling", "index"], | |||
["index", "internal", "parent", "sibling"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Put these in alphabetical order so it's easier to read them from left to right
["type"], | ||
["object"] | ||
["object"], | ||
["unknown"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The documentation for the order
rule added examples of all group types except for unknown
, so now that it's clear what each group type does, rather than have unknown
mixed into the Internal Imports
group, move it to its own group. Skimming through the ESLint plugin code doesn't reveal any obvious details either.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Pull Request Checklist
package.json
andpackage-lock.json
following Semantic VersioningChanges Included
1.2.1
import/order
rule so theunknown
import type now fills the last import group slot