-
Notifications
You must be signed in to change notification settings - Fork 40
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
semver + CHANGELOG.md #16
Comments
What is the problem with the version ? 0.8.1 is correct |
It doesn't follow semver, there were breaking API changes, so it should be at 1.x now (after the breaking change). |
hey @taoeffect what break? |
Err, actually... I might be wrong:
Somewhere between 0.6 and 0.8.1 dollar signs were introduces into the names of all the function calls. |
Anywho, I hope that wasn't a coincidence and you guys are actually planning on following semver. A changelog would still be nice (it's time consuming to was through the commits to figure out whether something serious changed or not). |
I usually follow semver, ESClass (the dependency) had deprecated non I also usually use a CHANGELOG, but since I forked this lib and refactored it a lot, I forgot to keep track of the changes :) (eg: https://github.com/pocesar/ES5-Class/blob/master/CHANGELOG.md) |
need to implement a proper CHANGELOG and report every breaking change (end user and implementors) |
Could you please start tracking changes and follow semver?
The text was updated successfully, but these errors were encountered: