-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Is there a changelog somewhere? #32
Comments
I can see that there is now a version 3.0.0 but also can't find a changelog. Assuming semantic versioning, I would expect to see breaking changes, but I don't have any idea what these may be. |
There isn't a changelog yet. v3 should be compatible with v2 for most common cases and it fixes some issues. But the truth is that v2 was missing a lot of tests so I wanted this to be a conscious decision. |
Thank you for the response José. In my application testing I'm not seeing any breaking changes. Proper use of semantic versioning does make it much easier for those of us using your library in assessing impact to our code bases. |
@ChrisWestcottUK understand and I use semantic versioning with all my libraries.. This is what it has been fixed between v2 and v3: https://github.com/jfromaniello/url-join/pull/35/files IMHO this should have been a patch actually, but these things where broken for so long that I was afraid other people where relying on broken/undesired behavior. my apologies |
@jfromaniello no need to apologise and thank you for providing the diff here. |
Is there a changelog or release notes somewhere? I don't know if it's safe to upgrade from 1.1.0 to 2.0.2. Thanks.
The text was updated successfully, but these errors were encountered: