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

Prevent self hosted jobs to run on forks. #5983

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

Ouss4
Copy link
Contributor

@Ouss4 Ouss4 commented Dec 6, 2021

Summary

Run jobs that require self hosted runners only on the main repository.
#5521 (comment)
adafruit#11

Impact

Forks only.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
@CLAassistant
Copy link

CLAassistant commented Dec 6, 2021

CLA assistant check)
All committers have signed the CLA.

@igrr igrr requested a review from tomassebestik December 7, 2021 12:41
@igrr
Copy link
Member

igrr commented Dec 7, 2021

Does this mean that we can't test external contributions on self-hosted runners at all?

@Ouss4
Copy link
Contributor Author

Ouss4 commented Dec 7, 2021

We can as long as they open a pull request in this repo. In their forks, it won't work.

@igrr
Copy link
Member

igrr commented Dec 7, 2021

Got it, thanks for explaining. LGTM then!

@me-no-dev me-no-dev merged commit 3750b14 into espressif:master Dec 14, 2021
@Ouss4 Ouss4 deleted the selfhosted branch December 21, 2021 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants