Skip to content

chore(catalog): re-pin shared verify action to merged .github main - #3

Merged
zircote merged 1 commit into
mainfrom
chore/repin-catalog-updater-merged-sha
Jun 23, 2026
Merged

chore(catalog): re-pin shared verify action to merged .github main#3
zircote merged 1 commit into
mainfrom
chore/repin-catalog-updater-merged-sha

Conversation

@zircote

@zircote zircote commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Re-pins catalog-admission's shared verify action from the transient #16 branch commit to the merged attested-delivery/.github main SHA b74df46, which includes the engine fixes from .github #16 and #17 (entry-scoped re-pin, PR-number selector, slug normalization, ref insertion).

This removes ccp main's dependency on the (now-mergeable) #16 feature branch commit. actionlint + pin-check clean; the cross-repo verify step was exercised green via workflow_dispatch on a prior branch (no-op at zero external entries).

Point catalog-admission's verify step at the merged attested-delivery/.github
main SHA (b74df46), which includes the engine fixes from .github #16 and #17.
Replaces the transient pin to #16's branch commit.
Copilot AI review requested due to automatic review settings June 23, 2026 00:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Re-pins the shared plugin-catalog-update verify action used by the catalog-admission workflow from a transient PR branch SHA to the merged attested-delivery/.github main commit SHA, removing reliance on the former feature-branch pin.

Changes:

  • Update uses: reference for the shared verify action to b74df46cd6e64777bd557a41c6f75a933cbc9617 (merged .github main).
  • Remove now-obsolete comments referring to the previous PR-branch pin.

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

@zircote
zircote merged commit 4cd5ac7 into main Jun 23, 2026
21 checks passed
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