Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Only run the build-contributor-pr workflow on forks #18082

Merged
merged 1 commit into from
Feb 19, 2021

Conversation

sarentz
Copy link
Contributor

@sarentz sarentz commented Feb 19, 2021

This patch has a better check to see if the PR is running on a pull request that came from a fork. (The current workflow does not trigger at all.)

if: github.event.pull_request.head.repo.full_name != github.repository

The github.repository is always the original repo, in our case mozilla-mobile/fenix while github.event.pull_request.head.repo points to the repository where the PR is originating from, which for contributor PRs can never be mozilla-mobile/*.

Let's hope this works 🤞

@sarentz sarentz requested review from a team as code owners February 19, 2021 15:03
@firefoxci-taskcluster
Copy link

No Taskcluster jobs started for this pull request
The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.

Copy link
Contributor

@st3fan st3fan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, let's give it a try!

@st3fan st3fan merged commit 1989d20 into mozilla-mobile:master Feb 19, 2021
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this pull request Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants