Skip to content

Revert "Merge pull request #133 from Keyfactor/replace-update-catalog…#135

Merged
bhillkeyfactor merged 1 commit into
mainfrom
revert-update-catalog-on-main
May 15, 2026
Merged

Revert "Merge pull request #133 from Keyfactor/replace-update-catalog…#135
bhillkeyfactor merged 1 commit into
mainfrom
revert-update-catalog-on-main

Conversation

@bhillkeyfactor
Copy link
Copy Markdown
Contributor

…-validation-pr-flow"

This reverts merge commit d62347a, returning .github/workflows/update-catalog.yml on main to its pre-#133 state.

#133 landed the validation gate and PR-based flow on main, but every integration repo's caller pins to @v5 (see starter.yml:141 on the v5 branch), so the merge had no operational effect. The same change has been forward-ported to v5 via a separate PR; that's the one that actually changes catalog behavior.

This revert keeps main aligned with the convention reflected in recent history (changes flow from version branches into main), rather than leaving main ahead of v5 with a change that bypassed the version-branch workflow.

No callers, no downstream effect — purely a history-hygiene revert.

…-validation-pr-flow"

This reverts merge commit d62347a, returning .github/workflows/update-catalog.yml
on main to its pre-#133 state.

#133 landed the validation gate and PR-based flow on main, but
every integration repo's caller pins to @v5 (see starter.yml:141
on the v5 branch), so the merge had no operational effect. The
same change has been forward-ported to v5 via a separate PR;
that's the one that actually changes catalog behavior.

This revert keeps main aligned with the convention reflected in
recent history (changes flow from version branches into main),
rather than leaving main ahead of v5 with a change that bypassed
the version-branch workflow.

No callers, no downstream effect — purely a history-hygiene revert.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bhillkeyfactor bhillkeyfactor merged commit 666e1bf into main May 15, 2026
1 of 3 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.

1 participant