Git Product home page Git Product logo

Comments (12)

bnoordhuis avatar bnoordhuis commented on May 21, 2024

No objections from me.

To prevent this from happening again, we might want to bump the minor and reset the is_release thing if a new API is added, thus indicating which version needs to be the next one.

The downside I see with that is that if two people add new APIs and neither is paying attention, then the minor gets bumped twice. Someone still needs to do a manual check, might as well be the person doing the release.

On the other hand, releases happen more frequently than new APIs so it might an acceptable trade-off. I'll go along with whatever you and @piscisaureus agree on.

from libuv.

saghul avatar saghul commented on May 21, 2024

Related to this, we should add the "versionadded" sphinx thingy for new APIs: #60

from libuv.

misterdjules avatar misterdjules commented on May 21, 2024

@saghul Is the 1.0.2 release going to be re-released so that it doesn't contain this new API?

from libuv.

saghul avatar saghul commented on May 21, 2024

@misterdjules I'd say no, "damage" is done. It's a new API, we didn't break an existing one, so I can live with it. It's also documented: http://docs.libuv.org/en/v1.x/loop.html#c.uv_loop_configure

So, if the SemVer police comes by, I'll talk to them :-)

from libuv.

bnoordhuis avatar bnoordhuis commented on May 21, 2024

I agree. You can't re-release a release; once people have downloaded it, it's out there. Changing it after the fact only leads to confusion ("hey, why does the checksum no longer match?").

from libuv.

saghul avatar saghul commented on May 21, 2024

Ok, #58 landed, which I wanted to include for this release. Any objections to making a 1.1.0 release tonight? Since I'm there I might as well release a 0.10.x.

from libuv.

bnoordhuis avatar bnoordhuis commented on May 21, 2024

@saghul Go for it.

from libuv.

othiym23 avatar othiym23 commented on May 21, 2024

if the SemVer police comes by, I'll talk to them :-)

Here we are! 🚓

You can't re-release a release; once people have downloaded it, it's out there. Changing it after the fact only leads to confusion ("hey, why does the checksum no longer match?").

Seconded. This is a big part of the reason why npm moved to not allowing force-republishing.

Also, minor version bumps should be a little more forgiving than major version bumps: you're only adding new APIs, not making changes that will affect downstream consumers unless they choose to opt into them (knowingly or otherwise).

🚔

from libuv.

sam-github avatar sam-github commented on May 21, 2024

Maybe the 1.0.2 release should be deleted, though, since it isn't 1.0.2, by semver?

Deleted from here: https://github.com/libuv/libuv/releases and perhaps even the tag, though that might live on in forks.

Are there tarballs? I can't find them, but if they are somewhere, the broken 1.0.2 should be deleted from there, too, IMO.

from libuv.

bnoordhuis avatar bnoordhuis commented on May 21, 2024

I disagree. It would be a different story if the 1.0.2 release contained a serious vulnerability that puts people at risk, but we're just talking about new functionality that was accidentally introduced in a patch release. Unfortunate, perhaps, but ultimately no big deal.

from libuv.

saghul avatar saghul commented on May 21, 2024

I'm with Ben. 1.0.2 just contains a new API, which can be version checked
just fine. Semantically it should have been 1.1.0, but the change is simple
enough not to overcomplicate things.
On Dec 23, 2014 11:12 PM, "Ben Noordhuis" [email protected] wrote:

I disagree. It would be a different story if the 1.0.2 release contained a
serious vulnerability that puts people at risk, but we're just talking
about new functionality that was accidentally introduced in a patch
release. Unfortunate, perhaps, but ultimately no big deal.


Reply to this email directly or view it on GitHub
#57 (comment).

from libuv.

saghul avatar saghul commented on May 21, 2024

I just released 1.1.0 (and 0.10.31) so, I'll close this and try not to forget next time!

@othiym23 sorry sir, it won't happen again, I promise! 😄

from libuv.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.