-
Notifications
You must be signed in to change notification settings - Fork 993
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
Commas in version-ranges make input invalid (breaking) #3964
Comments
Without commas, the string within the Are you agree with it, @conan-io/barbarians?
|
Yes, remove them as soon as possible from the docs. Then, introduce a deprecation warning, telling users to remove comas. |
Updated documentation by removing commas. |
Look into 13defc0. Would this message be fine? |
Version ranges with commas will be invalid in v2.0 and will throw an exception. |
Version ranges in |
I'm trying things here (https://semver.npmjs.com/) and in fact, commas are never used (we remove them in our code). Anyway, we should keep the possibility of having one comma (it is documented), but remove it for Conan 2.0.
For <2.0, it would be enough to remove those commas from the documentation: https://docs.conan.io/en/latest/mastering/version_ranges.html
Originally posted by @jgsogo in #3898
The text was updated successfully, but these errors were encountered: