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
fix: stop pinning plugins with prerelease versions #5554
Conversation
This pull request adds or modifies JavaScript ( |
packages/build/src/utils/semver.js
Outdated
export const isPrerelease = function (version) { | ||
// Prerelease versions are excluded from ranges unless `includePrerelease` is | ||
// set to `true`. | ||
return !semver.satisfies(version, '>=0.0.0', { includePrerelease: false }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you use semver.prerelease()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You prefer a simpler and more expressive method? FINE, whatever.
(This method is not in the docs, so I had to get creative. Thanks!)
Done in adc65d4.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for fixing!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for sorting this. I'll approve, though I'm not sure if the failing tests are relevant
2701063
[0m[31m[1m>[22m[39m[90m 1 |[39m [32m"json"[39m[0m␊ | ||
[0m [90m |[39m [31m[1m^[22m[39m[31m[1m^[22m[39m[31m[1m^[22m[39m[31m[1m^[22m[39m[31m[1m^[22m[39m[31m[1m^[22m[39m [31m[1m👈🏽 [95mtype[31m must be object[22m[39m[0m` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing
Summary
Fixes COM-566.