Skip to content

Restore main to released state; keep only the Claude workflow files - #844

Merged
erikdarlingdata merged 2 commits into
mainfrom
revert/accidental-dev-promotion
Jul 25, 2026
Merged

Restore main to released state; keep only the Claude workflow files#844
erikdarlingdata merged 2 commits into
mainfrom
revert/accidental-dev-promotion

Conversation

@erikdarlingdata

Copy link
Copy Markdown
Owner

Summary

  • Fixes a mistake in Add Claude Code PR review workflows to main (activation) #843: that PR's head branch was based on dev, so squash-merging it promoted ALL of dev's unreleased work to main, not just the two Claude workflow files it was meant to carry
  • This PR reverts both resulting commits (the accidental promotion and the bundle rebuild it triggered), restoring main to the released state at 9e912c2, then re-adds only the two workflow files
  • Verified: git diff 9e912c2 HEAD --stat on this branch shows exactly .github/workflows/claude-code-review.yml and .github/workflows/claude.yml, +121 lines, nothing else
  • This branch is based on main, so the PR diff is exactly the revert plus the two files; the bundle CI will restamp Install-All/DarlingData.sql from the restored sources on merge

Part of #841.

Test Plan

  • The 4 required SQL Server checks pass (they run the released sql-tests.yml against the released sources, the combination that was green at the July 4 release)
  • The source-branch guard check fails as expected (head is not dev); it is not a required check
  • After merge: main matches 9e912c2 plus the two workflow files, and the regenerated bundle matches released sources

🤖 Generated with Claude Code

erikdarlingdata and others added 2 commits July 25, 2026 11:51
PR #843 was meant to land only the two Claude workflow files on main,
but its head branch was based on dev, so the squash merge promoted all
of dev's unreleased work. This restores main to the released state
(9e912c2), including the regenerated bundle. The workflow files are
re-added separately in the next commit.
Adds automatic Claude code reviews on pull requests into dev, plus an
interactive workflow that responds to @claude mentions in issues and PR
comments (also the way to request reviews on fork PRs, which cannot see
repo secrets). Both authenticate with the CLAUDE_CODE_OAUTH_TOKEN secret.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@erikdarlingdata
erikdarlingdata merged commit 036f72f into main Jul 25, 2026
4 of 5 checks passed
@erikdarlingdata
erikdarlingdata deleted the revert/accidental-dev-promotion branch July 25, 2026 15:56
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