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

Bypass tag check in dry run (#35428) #35470

Closed

Conversation

cipolleschi
Copy link
Contributor

Summary

When dry-run in stable branch, the tag already exists. We are bypassing the tag-existence check when in a dry-run to have the CI pass

Changelog

[General] [Fixed] - Bypass tag version check when dry-run

Test Plan

CircleCI
It worked for RC1 and RC2

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Nov 25, 2022
@cipolleschi cipolleschi changed the title [LOCAL] bypass tag check in dry run (#35428) Bypass tag check in dry run (#35428) Nov 25, 2022
@cipolleschi cipolleschi force-pushed the cipolleschi/skip_version_tag_check_in_dry_run branch from e0ddfc8 to dcf08b3 Compare November 25, 2022 08:55
@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@analysis-bot
Copy link

analysis-bot commented Nov 25, 2022

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: f5e5274
Branch: main

@analysis-bot
Copy link

analysis-bot commented Nov 25, 2022

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,103,561 +0
android hermes armeabi-v7a 6,471,886 +0
android hermes x86 7,521,203 +0
android hermes x86_64 7,380,081 +0
android jsc arm64-v8a 8,968,399 +0
android jsc armeabi-v7a 7,699,505 +0
android jsc x86 9,030,706 +0
android jsc x86_64 9,508,749 +0

Base commit: f75b92a
Branch: main

@cipolleschi cipolleschi force-pushed the cipolleschi/skip_version_tag_check_in_dry_run branch from dcf08b3 to 829465a Compare November 25, 2022 11:44
@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@cipolleschi cipolleschi force-pushed the cipolleschi/skip_version_tag_check_in_dry_run branch from 829465a to 846e460 Compare November 25, 2022 15:26
@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @cipolleschi in 8b8f4f3.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Nov 25, 2022
@cipolleschi cipolleschi deleted the cipolleschi/skip_version_tag_check_in_dry_run branch March 27, 2023 13:26
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
Summary:
When dry-run in stable branch, the tag already exists. We are bypassing the tag-existence check when in a dry-run to have the CI pass

## Changelog

[General] [Fixed] - Bypass tag version check when dry-run

Pull Request resolved: facebook#35470

Test Plan:
CircleCI
It worked for RC1 and RC2

Reviewed By: cortinico

Differential Revision: D41529648

Pulled By: cipolleschi

fbshipit-source-id: d4d7f5534f86c2cf10b05e0d4cab950e4902d8df
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants