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

Do not attempt to comment on PRs when on forks #36316

Closed
wants to merge 2 commits into from
Closed

Conversation

cortinico
Copy link
Contributor

Summary

I'm gating this workflow to run only upstream as I'm receiving notifications that it fails on my fork. We don't want to run this workflow on forks at all hence we can add a if: there.

Changelog

[INTERNAL] - Do not attempt to comment on PRs when on forks

Test Plan

If Github Actions UI is green, we're good to go

@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 Feb 27, 2023
@Pranav-yadav
Copy link
Contributor

@cortinico can you also add this conditional to the rest of the GitHub workflows? Provided @SlyCaptainFlint also agrees.
I had disabled the actions completely in my fork's settings previously.

@facebook-github-bot
Copy link
Contributor

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

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,472,449 -1,014
android hermes armeabi-v7a 7,794,817 -1,144
android hermes x86 8,948,319 -1,196
android hermes x86_64 8,805,590 -1,015
android jsc arm64-v8a 9,106,493 -24
android jsc armeabi-v7a 8,303,676 -179
android jsc x86 9,157,267 -218
android jsc x86_64 9,416,316 -35

Base commit: 14ab76a
Branch: main

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Feb 28, 2023
@facebook-github-bot
Copy link
Contributor

@cortinico merged this pull request in d56bd11.

@cortinico cortinico deleted the cortinico-patch-1 branch April 24, 2023 14:58
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
Summary:
I'm gating this workflow to run only upstream as I'm receiving notifications that it fails on my fork. We don't want to run this workflow on forks at all hence we can add a `if:` there.

## Changelog

[INTERNAL] - Do not attempt to comment on PRs when on forks

Pull Request resolved: facebook#36316

Test Plan: If Github Actions UI is green, we're good to go

Reviewed By: cipolleschi

Differential Revision: D43657315

Pulled By: cortinico

fbshipit-source-id: 8fb260d142a43375bd9a3b10eac235550a6aecb0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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