-
Notifications
You must be signed in to change notification settings - Fork 24.9k
Update test-release-local script
#54385
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
Conversation
|
@j-piasecki has exported this pull request. If you are a Meta employee, you can view the originating Diff in D86105703. |
Summary: Changelog: [Internal] `test-release-local` shouldn't be changing the hermes version stored in `version.properties` as this script should be called on a cut branch with hermes already bumped. Differential Revision: D86105703
b215d20 to
ff998b4
Compare
test-release-local script
ff998b4 to
4172ecd
Compare
Summary: Changelog: [Internal] `test-release-local` shouldn't be changing the hermes version stored in `version.properties` as this script should be called on a cut branch with hermes already bumped. Reviewed By: cipolleschi Differential Revision: D86105703
Summary: Pull Request resolved: facebook#54385 Changelog: [Internal] `test-release-local` shouldn't be changing the hermes version stored in `version.properties` as this script should be called on a cut branch with hermes already bumped. Reviewed By: cipolleschi Differential Revision: D86105703
4172ecd to
3e6546c
Compare
|
This pull request has been merged in 30370f6. |
|
This pull request was successfully merged by @j-piasecki in 30370f6 When will my fix make it into a release? | How to file a pick request? |
Summary: Pull Request resolved: #54385 Changelog: [Internal] `test-release-local` shouldn't be changing the hermes version stored in `version.properties` as this script should be called on a cut branch with hermes already bumped. Reviewed By: cipolleschi Differential Revision: D86105703 fbshipit-source-id: 5a1edd9f7a6cd756521749c2b0023ea683dadf82
|
This pull request was successfully merged by @j-piasecki in e23a285 When will my fix make it into a release? | How to file a pick request? |
Summary:
Changelog: [Internal]
test-release-localis not a ES6 module and it should use require instead of import.Differential Revision: D86105703