-
Notifications
You must be signed in to change notification settings - Fork 509
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
Acceptance+handling of partial versions in -
ranges not documented
#76
Comments
So... on this one, I'm a bit torn. I added the tests because I wanted to at least know if the behavior ever changes, just in case it would break anyone. But, I'm tempted to say that a Out of curiosity, are you writing a clean-room implementation of this for? I've been meaning to document the range syntax as a proper specification eventually, maybe this could be a good justification for doing that, along with a comprehensive suite of tests etc. |
Your call obviously. I basically just care about the syntax+semantics getting adequately doc'd/spec'd, regardless of what exactly the spec'd syntax+semantics are.
I'm writing a Python library that parses various ecosystems' dependency specifications (e.g. |
Fixed on v4, #99 |
The only example of the
-
range operator in the README shows it being used with full versions, but some testcases have it also being used with incomplete versions, in particular with novel handling of incomplete upper-bounds: https://github.com/isaacs/node-semver/blob/v2.2.1/test/index.js#L491This should be documented.
It's funny, the things you run into when trying to write a clean-room implementation...
The text was updated successfully, but these errors were encountered: