Skip to content

[cbr79] Fix build check for forked repos#962

Open
bmastbergen wants to merge 2 commits intociqcbr7_9from
{bmastbergen}_ciqcbr7_9
Open

[cbr79] Fix build check for forked repos#962
bmastbergen wants to merge 2 commits intociqcbr7_9from
{bmastbergen}_ciqcbr7_9

Conversation

@bmastbergen
Copy link
Collaborator

@bmastbergen bmastbergen commented Mar 11, 2026

Here is a test PR with the same workflow from a forked repo. It ran the build check just fine:
#961

commit cd6249ce75127adf489d97ebe15ee952a77aeb56
Author: Brett Mastbergen <bmastbergen@ciq.com>
Date:   Wed Mar 11 12:08:22 2026 -0400

    github actions: handle PRs from forked repos

    Detect whether the PR is from a fork and clone from the appropriate
    source. Use env vars for GitHub context values to avoid script injection.
    Explicitly name the clone destination to avoid directory mismatch if a
    fork has a different repo name.

    Note: the standard checkout action is not used here because the git
    version in this container is too old.

 .github/workflows/build-check_x86_64.yml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)
commit dc93d458821e12022e69c5d6100142fd97621445
Author: Brett Mastbergen <bmastbergen@ciq.com>
Date:   Wed Mar 11 12:21:32 2026 -0400

    github actions: use shallow clone for kernel checkout

 .github/workflows/build-check_x86_64.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Detect whether the PR is from a fork and clone from the appropriate
source. Use env vars for GitHub context values to avoid script injection.
Explicitly name the clone destination to avoid directory mismatch if a
fork has a different repo name.

Note: the standard checkout action is not used here because the git
version in this container is too old.
@bmastbergen bmastbergen requested a review from a team March 11, 2026 17:04
@github-actions
Copy link

🤖 Validation Checks In Progress Workflow run: https://github.com/ctrliq/kernel-src-tree/actions/runs/22964980625

@github-actions
Copy link

Validation checks completed successfully View full results: https://github.com/ctrliq/kernel-src-tree/actions/runs/22964980625

@PlaidCat PlaidCat requested a review from a team March 11, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants