Skip to content

pkg-build-pr-check: Fix repo/ref when fork PR#293

Open
simonbeaudoin0935 wants to merge 1 commit intoqualcomm:developmentfrom
simonbeaudoin0935:fix/upstream-pr-build-workflow
Open

pkg-build-pr-check: Fix repo/ref when fork PR#293
simonbeaudoin0935 wants to merge 1 commit intoqualcomm:developmentfrom
simonbeaudoin0935:fix/upstream-pr-build-workflow

Conversation

@simonbeaudoin0935
Copy link
Contributor

The pkg-build-pr-check workflow triggers a build of the associated pkg-repo of the upstream project.
Before this fix, the workflow logic did not handle properly when the PR was from a fork of the upstream project as opposed to a branch from the repo itseld. Now, the proper repo and ref are forwarded to the reusable workflow.

Note that for this to work, the forked repo needs to be public. This should logically always be the case because our upstream repos are public to begin with, and a fork from a public repo has not choice but to be public as well.

When the PR is from a fork of the upstream project,
the pr-build-pr-check workflows did not handle that
correctly. Now, the proper repo and ref are forwarded
to the reusable workflow_

Signed-off-by: Simon Beaudoin <sbeaudoi@qti.qualcomm.com>
@quic-mtharu
Copy link
Contributor

Has this workflow been verified? Please ensure it is verified in the staging environment first and then provide the details of the verified workflow here.

qualcomm-linux-stg/fastrpc

@simonbeaudoin0935
Copy link
Contributor Author

Has this workflow been verified? Please ensure it is verified in the staging environment first and then provide the details of the verified workflow here.

This is a fix to make the pkg-build pass in the case of a pr coming from a fork.

I did not know we had a staging version of this repo though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants