Add sort by length option #224
Conversation
|
Would love to have this! |
|
I also would love to have this. When it will be on the prod guys? |
|
Had to update the branch, think it needs reaproving |
|
If this gets merged prettier can be renamed prettiest :) |
|
+1 for this, could we merge it? |
|
Gonna need someone to approve and merge it, keep coming back to it approved with merge blocks |
Co-authored-by: annes <[email protected]>
|
@annesnour03 can you approve so I can get this put away |
|
I still cant merge it @annesnour03 so idk if you can if not... it waits even more XD |
|
|
|
@ayusharma @byara come to our rescue please! |
|
Any news on this :)? @ayusharma & @byara |
|
@ayusharma @byara please add this |
|
1 year later XD |
|
@ayusharma Could someone approve / merge this again please |
|
👀 |
|
So sad that we've been waiting for this feature for so long and nobody can merge it! |
|
I think react 20 will be published before this pr gets merged |
src/utils/get-sorted-nodes.ts
Outdated
| importOrderSeparation, | ||
| importOrderSortSpecifiers, | ||
| importOrderGroupNamespaceSpecifiers, | ||
| importOrderGroupNamespaceSpecifiers,importOrderSortByLength |
There was a problem hiding this comment.
| importOrderGroupNamespaceSpecifiers,importOrderSortByLength | |
| importOrderGroupNamespaceSpecifiers, | |
| importOrderSortByLength |
There was a problem hiding this comment.
should be in now, im gonna go through and check it still works as the merge may not be ok given how long its been
There was a problem hiding this comment.
If you're still around, latest commit should fix any tests, update docs / types and work
There was a problem hiding this comment.
looks good to me, I really appreciate the effort. Hopefully some maintainer is able to merge this wonderful PR! 😉
There was a problem hiding this comment.
I think if you approve the pr again, we can get it in. Unless Github is leading me astray telling me it needs you to review.. if not the wait goes on 😅
|
love this! |
|
look this: prettier-plugin-sort-imports,can do this work |
|
Everyone clearly loves this configuration option, so why on earth do the maintainers steadfastly refuse to merge it? I just can’t figure it out. |
|
Any news on this :)? @ayusharma & @byara |
|
Hey @vladislavarsenev could you please take a look at this? |
|
Thank you for such a clean and useful PR! it's time to merge, this PR is old enough now |
|
omg its in 2 years! thanks @vladislavarsenev people rejoice |
|
we are potentially so back |
15 participants
Adds the option to be able to sort within import groups by length mentioned #76, choices are
ascshortest to longestdesclongest to shortestnullno length rule appliedAfter length imports are sorted alphabetically
I have an internal fork with this implemented already so this doesn't affect me, but this is the general setup I have been using