Skip to content

ci: pin checkout SHA; explicitly checkout z-shell/zd in reusable workflow#65

Open
ss-o wants to merge 1 commit into
mainfrom
ci/fix-reusable-checkout
Open

ci: pin checkout SHA; explicitly checkout z-shell/zd in reusable workflow#65
ss-o wants to merge 1 commit into
mainfrom
ci/fix-reusable-checkout

Conversation

@ss-o
Copy link
Copy Markdown
Member

@ss-o ss-o commented May 19, 2026

Summary

  • Pin actions/checkout to a full commit SHA in the reusable test-native.yml workflow
  • Explicitly checkout z-shell/zd (instead of using github.sha which is not cloneable as a branch ref when called from external repos via workflow_call)

Test plan

  • Verify the reusable workflow can be called from z-shell/zi with workflow_call without checkout errors
  • Confirm test-native.yml passes end-to-end

Copilot AI review requested due to automatic review settings May 19, 2026 06:10
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the reusable native test workflow to be safer/more deterministic by pinning actions/checkout and ensuring the workflow checks out the intended repository when invoked via workflow_call.

Changes:

  • Pinned actions/checkout to a full commit SHA in test-native.yml.
  • Updated checkout step to explicitly clone z-shell/zd (instead of implicitly using the caller repo in workflow_call scenarios).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +47 to +49
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
with:
repository: z-shell/zd
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