-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add a changelog #262
Comments
I am happy to add the 2 files. For now the changelog would list changes from 2.8.0. I'm not sure if/when the other log of changes would be added. I currently maintain a log on the website: release notes I'll also take a look at this SemVer standard to see where we can improve. |
Thanks! The idea behind SemVer is pretty simple actually. The version string consists of three parts:
In case of jPlayer it fails to increase the |
Aha... So you want us to make a MAJOR version increment just because the repository structure changed? jPlayer only had a few minor tweak - PATCH - changes and then included the new features when I incremented the MINOR version in 2.8.0. The changes to the repository were to make all the files available in one place and to make it easy for novices to download the repo and for it to work without installing any 3rd party apps, like node or bower. I'll not comment any more on this before I get round to reading the doc. |
Plus, we actually already follow those rules... http://jplayer.org/download/ The difference being, my point of view is when jPlayer changes... Your point of view is when the NPM package changes. jPlayer remains backward compatible, it is the package that has changed in structure. |
The main topic of this issue has been addressed and pushed to the v2.8 branch. |
If you are separating this things (i.e. version of the jPlayer plugin and version of the GitHub repository) then GitHub repository should have a versioning system compatible with SemVer, because Bower and npm are using the version string to update dependencies. In my case, my project was broken after a minor routine dependency update procedure (and it shouldn't really happen) because:
|
Thanks again @slavafomin for explaining. I will talk with @maboa tomorrow about raising the major version to 3. To me it is just another number, but someone might think it is a groundbreaking new release of jPlayer. I have this nagging feeling that I've already broken bower for the 2.x series and I will not have the time to fix that retroactively. It would be rather complicated and I doubt my git skills are up to it. Also, increasing the major version now is already too late, except that it might be more obvious how I messed up to veterans like yourself. NPM should actually be fine, because I never published it before 2.8.0, but I suppose people may have npm installed direct from the jplayer git repo - so ok, I messed that up too. Plus... This changelog is going to double my release notes writeup. Think I'll have to dump the webpage and then point to the changelog... Not sure what I'll do there actually. Maybe link between the old and new for the time being. Just trying to keep DRY. |
Just a thought to keep changelog DRY: You can create a changelog in Markdown format in this repository and move all changelog details in it from your web page. Then you can edit this file when you introduce any changes. On your webpage you can actually load this |
As far as jPlayer is not really consistent with SemVer standard I think it will be a good idea to introduce a
changelog.md
file with the list of changes introduced in new versions. This will really help with migration. I will lock my project to the current version of jPlayer and will update it manually. Changelog will help a lot. Or at least we can usemigration.md
file with the list of changes that break compatibility.Thanks!
The text was updated successfully, but these errors were encountered: