Skip to content

.github/workflows: skip jobs in forks - #90

Open
Loïc Minier (lool) wants to merge 1 commit into
qualcomm-linux:mainfrom
lool:workflow-fixes
Open

.github/workflows: skip jobs in forks#90
Loïc Minier (lool) wants to merge 1 commit into
qualcomm-linux:mainfrom
lool:workflow-fixes

Conversation

@lool

Copy link
Copy Markdown
Contributor

Four workflows self-trigger on push, pull_request, or schedule and
fail or misbehave when this repo is forked: build-test needs
DEBUSINE_TOKEN, the container build pushes to
ghcr.io/qualcomm-linux, the preflight checks are Qualcomm-specific,
and stale-issues would comment on a fork's own issues. Guard each
job on the repository owner.

Four workflows self-trigger on push, pull_request, or schedule and
fail or misbehave when this repo is forked: build-test needs
DEBUSINE_TOKEN, the container build pushes to
ghcr.io/qualcomm-linux, the preflight checks are Qualcomm-specific,
and stale-issues would comment on a fork's own issues. Guard each
job on the repository owner.

Assisted-by: Claude Code:claude-opus-5
Signed-off-by: Loïc Minier <loic.minier@oss.qualcomm.com>

@basak-qcom Robie Basak (basak-qcom) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Most of this looks good but I'm unsure about one specific change. Question inline.

name: Run QC Preflight Checks
# Qualcomm-specific checks (commit email, copyright, repolinter) that are
# not meaningful in a fork.
if: github.repository_owner == 'qualcomm-linux'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I agree with the others. But if our aim is to accept PRs from forks, I think we do want this one, no?

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.

2 participants