-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Versioning support #2343
Comments
Same problem. |
What the f*** is this??? |
Y'all should calm down. A) your test suite caught them, and while annoying that's easy enough to not upgrade or make an issue about it, it's an open source project you can contribute to since it needs love and support. |
👋 |
nevertheless I agree with you @pirvudoru 👍 should have followed the semver strictly. unfortunately we are not quite sure at this point in time if v1 would be released in near future, from which it would be easier to maintain the semver policy |
@wasifhossain 0.10 is supposed to be stable. AMS is older than semver. There shouldn't be breaking changes with 0.10.x. Just like rake of old, think of 0.10.x and 10.x |
@pethron I think you and @pirvudoru may be looking at master https://github.com/rails-api/active_model_serializers/blob/bd40fe68f0ba7332a4b86c793149ebbcd207ca9f/.travis.yml the 0.10.x release comes from the 0-10-stable branch |
Expected behavior vs actual behavior
According to: https://semver.org/ #3:
I expect patches to not break the app.
But...

This library is a nightmare to work with when it comes to updating the version, and it has been this way since v 0.7.x (when we started using it - huge mistake).
The text was updated successfully, but these errors were encountered: