eslint-plugin-typescript-sort-keys icon indicating copy to clipboard operation
eslint-plugin-typescript-sort-keys copied to clipboard

Consider contributing features up to typescript-eslint?

Open JoshuaKGoldberg opened this issue 3 years ago • 5 comments

Hi! I'm a maintainer on typescript-eslint and just stumbled onto this plugin from a feature request on the typescript-eslint member-ordering rule (https://github.com/typescript-eslint/typescript-eslint/issues/4859). It looks like much of this plugin's functionality is already in member-ordering.

Is there anything in particular you'd want to be added to member-ordering that doesn't exist in it already?

JoshuaKGoldberg avatar Apr 25 '22 16:04 JoshuaKGoldberg

Hey @JoshuaKGoldberg Thanks for reaching out! There are many plugin users on npm, so I would expect all the of the options from interface and string-enum rules to be supported in member-ordering

infctr avatar Apr 25 '22 18:04 infctr

I can see that requiredFirst: true is not supported in member-ordering, is it?

infctr avatar Apr 25 '22 18:04 infctr

Ooh, it isn't, no. Now that I'm taking a deeper look I see the natural sorting option isn't either.

Are you up for filing an issue on typescript-eslint and/or sending a PR in? No worries if that's a sudden unexpected ask you don't have time for 😄, I can take on whatever you don't.

I'll also note our docs page isn't great right now and I plan on fixing it up soon: https://github.com/typescript-eslint/typescript-eslint/issues/4861

JoshuaKGoldberg avatar Apr 25 '22 19:04 JoshuaKGoldberg

@JoshuaKGoldberg Go ahead :)

infctr avatar Apr 26 '22 10:04 infctr

Just as a user looking between these two, the reason I'm choosing eslint-plugin-typescript-sort-keys is the auto-fixer

zachbryant avatar Aug 30 '22 21:08 zachbryant

Here’s the FR for auto fixer on official TS ESLint side: https://github.com/typescript-eslint/typescript-eslint/issues/2296

connorjs avatar Mar 23 '23 18:03 connorjs

Closing out my old issues for repos I no longer have context on. If anybody has a need for what this issue was asking about, I'd encourage them/you to file a new issue. Cheers!

JoshuaKGoldberg avatar Feb 17 '24 20:02 JoshuaKGoldberg