Skip to content

Use shared render action as PR merge gate#13

Draft
shielded-nate with Copilot wants to merge 1 commit into
add-flake-rendering-outputsfrom
copilot/add-flake-rendering-outputs
Draft

Use shared render action as PR merge gate#13
shielded-nate with Copilot wants to merge 1 commit into
add-flake-rendering-outputsfrom
copilot/add-flake-rendering-outputs

Conversation

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown

This updates PR render checks so merge acceptance is driven by the shared render action path, aligned with repository rendering behavior. The PR workflow now invokes the composite render action directly instead of maintaining a separate inline render pipeline.

  • Workflow gate alignment

    • .github/workflows/render.yml now uses ./.github/actions/render for the PR render job.
    • This makes the render check the canonical merge criterion via the reusable action.
  • Refactor of PR render workflow

    • Removed duplicated inline Nix install/cache/export and compile steps from the workflow.
    • Kept base_ref setup in the workflow so protocol conditional build logic remains available to the composite action.
  • Resulting execution path

    • PR renderability checks and shared render implementation now run through one maintained action surface.
- name: Set base ref
  run: |
    git show --format=%H --no-notes --no-patch "HEAD~1" -- |tee base_ref
    if ! ( grep -E '^[0-9a-f]{40}$' base_ref ); then exit 1; fi

- name: Compile ZIPs and Zcash Protocol Specification
  uses: ./.github/actions/render

Copilot AI changed the title Use shared render action for PR merge checks Use shared render action as PR merge gate Jun 17, 2026
Copilot AI requested a review from shielded-nate June 17, 2026 01:41
@shielded-nate shielded-nate changed the base branch from main to add-flake-rendering-outputs June 18, 2026 22:48
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