Skip to content
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

0.5.3 is not published to npm #459

Closed
kelset opened this issue Feb 25, 2021 · 10 comments
Closed

0.5.3 is not published to npm #459

kelset opened this issue Feb 25, 2021 · 10 comments
Labels

Comments

@kelset
Copy link

kelset commented Feb 25, 2021

Bug Description

According to the release tab, since Nov 2020 there should be a 0.5.3 release of Hermes available.

Problem is, according to npm it's not there.

Steps To Reproduce

  1. use yarn resolution to force hermes-engine to version 0.5.3
  2. run yarn install
  3. see yarn telling you 0.5.3 doesn't exist:

Couldn't find any versions for "hermes-engine" that matches "0.5.3"

The Expected Behavior

0.5.3 should be available via npm.

@Huxpro
Copy link
Contributor

Huxpro commented Feb 25, 2021

@kelset sorry for the confusion. The 0.5.3 was solely released to support the RN macOS arm64 (see #390) hence it's only published to the https://www.npmjs.com/package/hermes-engine-darwin

That being said, is there any use case other than RN macOS of 0.5.3 you are looking for? There shouldn't be any differences with 0.5.2 on other platform.

My assumption is that RN 0.63 declared 0.5.x so the most recent patch releases would be automagically selected (0.5.2 0.5.1 as for hermes-engine).

@Huxpro Huxpro added the release label Feb 25, 2021
@jcamins
Copy link

jcamins commented Feb 26, 2021

@Huxpro there doesn't seem to be a 0.5.2 release either, just a release candidate. Is 0.5.2-rc1 actually the final release?

@Huxpro
Copy link
Contributor

Huxpro commented Feb 26, 2021

@jcamins Oops, yeah you are right there is no0.5.2 and 0.5.2-rc1 was made for opting-in Proxy.

But my assumption still hold, RN 0.63 would pick whatever the most recent patch release is. There was no important addition after 0.5.1.

Does this break any tooling (that based on some assumption of the existence of them) or create any problems? Is this a well-established convention?

@jcamins
Copy link

jcamins commented Feb 26, 2021

@Huxpro I want to try using hermes in a project depending on Proxy. Since there is a non-prerelease version after the 0.5.2-rc1 release, I interpreted that to mean there is a stable version on the 0.5.x line with Proxy support. If 0.5.x with Proxy is not stable, I would expect there to be a 0.5.3-rc1 release instead.

@Huxpro
Copy link
Contributor

Huxpro commented Feb 26, 2021

@jcamins I see.

Since there is a non-prerelease version after the 0.5.2-rc1 release, I interpreted that to mean there is a stable version on the 0.5.x line with Proxy support.

There isn't. We marked it with -rc suffix so it can be opted in and it won't break non-Proxy user by accident in case the Proxy has bugs. But turns out it's already quite stable from the feedbacks at #33 so we enabled it by default at 0.7.0 and people wanting Proxy at RN 0.63 seem to be happy using 0.5.2-rc1.

That being said, I can see now why people may expect there is a stable 0.5.x release with Proxy enabled. The reason it was not made was not technical but:

  1. We assumed people wanting Proxy can find out 0.5.2-rc1 from Consider supporting Proxy #33 or the release notes.
  2. The focus of our work were already moved to support RN 0.64 with 0.7.x line. (We expected it to come a little sooner.)

@Huxpro
Copy link
Contributor

Huxpro commented Feb 26, 2021

I've updated the release notes of https://github.com/facebook/hermes/releases/tag/v0.5.2-rc1 to reflect the current situation more clearly.

@jcamins thanks for this valuable feedback!

@jcamins
Copy link

jcamins commented Feb 26, 2021

Could I request one additional update to https://github.com/facebook/hermes/releases/tag/v0.5.3 along the lines of "Note: this release is only for Darwin-ARM64. Please use v0.5.2-rc1 for other platforms"? This way anyone seeing 0.5.3 on top will be alerted to use 0.5.2-rc1.

@Huxpro
Copy link
Contributor

Huxpro commented Feb 26, 2021

@jcamins Absolutely. Thanks for the input!

@Huxpro
Copy link
Contributor

Huxpro commented Feb 26, 2021

@kelset is your concern same as @jcamins?
Feel free to close it if we agreed this is resolved ;)

@kelset
Copy link
Author

kelset commented Mar 1, 2021

👍

@kelset kelset closed this as completed Mar 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants