-
Notifications
You must be signed in to change notification settings - Fork 32
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
Update dependency tree for stylelint 14.0.1 #65
Comments
please update |
It seems @YozhikM is not very active on GitHub nowadays. Please note that I have created a fork that works:
|
Even I am facing this issue. |
For people following this discussion – there’s now talks of either adding some of those rules to Stylelint itself or creating another community plugin similar to stylelint-a11y here: stylelint/stylelint#6212. @ronilaukkarinen it’d be great to get your thoughts on this in particular! |
Thanks for the ping @thibaudcolas, I'll add my two cents. |
YozhikM/stylelint-a11y appears to be abandoned[1], and doesn't support stylelint versions past 13.0.0. GitHub user @ronilaukkarinen has created a fork to provide compatibility with newer stylelint releases. [1]: YozhikM/stylelint-a11y#65
Conclusion: Meantime YozhikM/stylelint-a11y was forked multiple times. As of now https://github.com/mechanical-ink/stylelint-a11y is not yet available as package on nodejs.com. The following seams to be active (recent activity): |
/CLOSE |
@christian-weiss Thanks for your investigation! Both @ronilaukkarinen/stylelint-a11y and @double-great/stylelint-a11y seem incompatible with stylelint 16. Do you know if the one from @mechanical-ink is compatible with stylelint 16? @YozhikM are you willing to add additional people to the npm package (https://www.npmjs.com/package/stylelint-a11y) so development can be taken over? @ronilaukkarinen What are your thoughts on this going forward? |
I can update it asap. |
Updated. 16 support in v1.2.8. |
@ronilaukkarinen Thanks! When using it with stylelint 16, I still get the following error: @mechanical-ink and @double-great can we agree on one single repository for maintaining this dependency going forward? |
Hmm, somehow my tests didn't cause any errors. Going to take another look at it. As for the other discussion, we're having the same in several issues. My take is here (tl;dr; I'm going to keep maintaining my own fork regardless). |
Forced to revert back for the time being, see this. v1.2.9 out. Let's continue the discussion in their proper issues. |
This plugin is no longer maintained, and I found I was disabling one of the three rules it added most of the time. @see YozhikM/stylelint-a11y#65
It seems dependencies are outdated. Please update.
The text was updated successfully, but these errors were encountered: