eslint-config-canonical icon indicating copy to clipboard operation
eslint-config-canonical copied to clipboard

Add `eslint-plugin-decorator-position`

Open Bosseskompis opened this issue 3 years ago • 1 comments

Thoughts on adding a rule for decorator position? Doesn't necessarily have to be eslint-plugin-decorator-position

Bosseskompis avatar Dec 25 '21 22:12 Bosseskompis

Not keen on adding a whole dependency just for one plugin. However, as an alternative, would you be open to contribute it to https://github.com/gajus/eslint-plugin-canonical ?

eslint-plugin-canonical is an unopinionated ESLint plugin that encourages community to add their own ESLint rules instead of releasing new packages just to publish a single rule.

gajus avatar Jan 03 '22 18:01 gajus

Suggesting porting @NullVoxPopuli rule to eslint-plugin-canonical if you'd like it as part of the rulset.

gajus avatar Jan 16 '23 18:01 gajus

something wrong with the dependency? I'm generally anti code duplication for the sake of "no dependencies". I don't think it's truthful, or well-intended -- but rather "click-bait-esque" phrasing to convince people "oh, this is light weight", or something.

NullVoxPopuli avatar Jan 16 '23 18:01 NullVoxPopuli

It is not about duplicate code – it is about that all of these single-rule packages becoming unmaintained the moment the maintainer loses interest in that particular rule or simply not being active enough to keep the package up to date. The whole reason eslint-plugin-canonical came into existence was because a few packages were lagging behind important updates.

gajus avatar Jan 16 '23 18:01 gajus

But I am happy to add anyone with a track record of open-source work to maintain eslint-plugin-canonical.

gajus avatar Jan 16 '23 18:01 gajus

That's fair. tbh, my hope is that once decorators move to Stage 4, prettier will take over responsibility of the type of formatting my package does

NullVoxPopuli avatar Jan 16 '23 18:01 NullVoxPopuli