stylelint-find-rules icon indicating copy to clipboard operation
stylelint-find-rules copied to clipboard

`extends` with `plugins` doesn't work

Open axten opened this issue 6 years ago • 8 comments

...so for now, removing this in favor of stylelint-find-new-rules

axten avatar Dec 09 '18 00:12 axten

@axten Is this related to https://github.com/alexilyaev/stylelint-find-rules/issues/9 specifically or something else is broken?

alexilyaev avatar Dec 09 '18 07:12 alexilyaev

I don't know... What I can say is that is was broken with version 0.8 0.7 was working fine.

I just link to an order.js with rules, like this

    extends: [
        './rules/order',
    ],

axten avatar Dec 09 '18 10:12 axten

@axten Can you please elaborate on your full use case?
What did you expect the results to be?

in v0.8.0 I've added support for extends:

How does it handle extends?

The rules of extends are added to the list of configured rules, thus, if a rule is covered in an extend config, it will not show up in the unused results.

So unless ./rules/order has an extends of it's own, this should work.
And by "work" I mean it should not show rules defined in the extends as being unused.

alexilyaev avatar Dec 11 '18 23:12 alexilyaev

@axten I've put in some work and now in v2.0.0 it fully supports extends of extends:
https://github.com/alexilyaev/stylelint-find-rules/releases/tag/v2.0.0

Would you mind giving it a try and letting me know if it works for you?

alexilyaev avatar Dec 11 '18 23:12 alexilyaev

cool, it's not broken anymore! UNUSED looks good

but INVALID is wrong:

INVALID: Configured rules that are no longer available:

RULE
order/order
order/properties-order

so 'stylelint-order' plugin is not considered correctly.

axten avatar Dec 12 '18 07:12 axten

Hmm, so it's time to support plugins :-)

On Wed, Dec 12, 2018, 09:45 axten <[email protected] wrote:

cool, it's not broken anymore! UNUSED looks good

but INVALID is wrong:

INVALID: Configured rules that are no longer available:

RULE order/order order/properties-order

so 'stylelint-order' plugin is not considered correctly.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/alexilyaev/stylelint-find-rules/issues/18#issuecomment-446492973, or mute the thread https://github.com/notifications/unsubscribe-auth/AAdU1WEiVKO6NKq7CgBkG5JAYRRnP9QRks5u4LQngaJpZM4ZJ4MI .

alexilyaev avatar Dec 14 '18 08:12 alexilyaev

maybe https://github.com/Donov4n/stylelint-find-new-rules can give you a hint... maybe its time to join forces

axten avatar Dec 14 '18 11:12 axten

Yeah, mentioned it at https://github.com/Donov4n/stylelint-find-new-rules/issues/2

And will check it as part of https://github.com/alexilyaev/stylelint-find-rules/issues/20

alexilyaev avatar Dec 16 '18 10:12 alexilyaev