Skip to content
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

Extend stylelint peerDep #56

Closed
wants to merge 1 commit into from
Closed

Conversation

calebeby
Copy link

@calebeby calebeby commented Jul 24, 2023

It looks like stylelint was updated in the devDependencies without also updating the peerDependencies. This means that while this project is set up and tested using stylelint 15, projects that use this project can't use stylelint 15; they have to use 14.

This change extends the stylelint peerDependency version range to include 15. An alternative path would be to change the peerDependency to only support stylelint 15, if you don't intend to continue testing with stylelint 14, although this would be a breaking change.

It looks like stylelint was updated in the devDependencies without also updating the peerDependencies. This means that while this project is set up and tested using stylelint 15, projects that use this project can't use stylelint 15 and have to use 14 (at least without warnings from npm). This change extends the stylelint peerDependency version range to include 15. An alternative path would be to change the peerDependency to _only_ support stylelint 15, if you don't intend to continue testing with stylelint 14, although this would be a breaking change.
@AlecRust
Copy link
Collaborator

Thanks @calebeby looks like I missed that.

I've bumped other dependencies and included this fix in #57, after which I think we can release v20.

@AlecRust AlecRust closed this Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants