Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Release candidate #15

Closed
dyladan opened this issue Mar 1, 2021 · 10 comments
Closed

Release candidate #15

dyladan opened this issue Mar 1, 2021 · 10 comments

Comments

@dyladan
Copy link
Member

dyladan commented Mar 1, 2021

Is it sufficient to put on the README for the API that 0.18 is the release candidate? Or do we want to release something like 1.0.0-rc.1?

Originally posted by @dyladan in open-telemetry/opentelemetry-js#1982 (comment)

@dyladan
Copy link
Member Author

dyladan commented Mar 1, 2021

/cc @open-telemetry/javascript-maintainers

@vmarchaud
Copy link
Member

I think we can tag it both on github and npm ?

@dyladan
Copy link
Member Author

dyladan commented Mar 1, 2021

I think we can tag it both on github and npm ?

To me it seems like there are a few places we could tag the current version as RC:

  • GitHub README
  • git tag (rc1 or similar?)
  • npm release tags (rc maybe?). I'm not sure how useful this is because when people do npm install @opentelemetry/api, they will get latest by default so marking it as rc is unlikely to be seen or noticed by anyone.

In addition to that, we could release a version like 1.0.0-rc.1 which follows semver conventions for a prerelease.

The question is: is it sufficient to tag 0.18 as our release candidate? Or do we need to release a semver prerelease version also?

@vmarchaud
Copy link
Member

looking at other SIGS:

Do we actually expect to run RC release ? If not i'm not sure making a full release would be that useful. Generally i would go with git tag + npm releases to tag it somewhere.

@legendecas
Copy link
Member

If we define such release as RC, I'd vote for semver like 1.0.0-rc.1 to explicitly say that this version is ready for a stable release. And people can start using ^1.0.0-rc.1 version range, which would get updated if we released stable 1.0.0.

@dyladan
Copy link
Member Author

dyladan commented Mar 2, 2021

This is why I wanted it to be the same as 0.18. If someone installs npm i @opentelemetry/api they will get 0.18.0, but using npm tags we can make it so if they explicitly install npm i @opentelemetry/api@rc they can get 1.0.0-rc.1 which will automatically be updated to 1.0.0. This means we can "safely" add breaking changes to the RC if absolutely required because users have explicitly opted into it.

@vmarchaud
Copy link
Member

I see, i think adding a note in the readme + a tag on github (that point to the same commit as 0.18.0) for historic purpose would be fine ?

@Flarna
Copy link
Member

Flarna commented Mar 11, 2021

This means we can "safely" add breaking changes to the RC

As far as I remember your version check in API doesn't include the pre-release part. Therefore I think breaking changes are no longer allowed in this phase.

@dyladan
Copy link
Member Author

dyladan commented Mar 12, 2021

This means we can "safely" add breaking changes to the RC

As far as I remember your version check in API doesn't include the pre-release part. Therefore I think breaking changes are no longer allowed in this phase.

That is correct. I didn't mean "safely" as in "it wont break users" but the only users who have the rc are ones who are explicitly opted in. I think we should try as hard as possible to avoid breaking.

@vmarchaud
Copy link
Member

I believe we can close this since we went GA

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants