Skip to content

chore: drop accidentally-merged review-approval feature work from main#317

Merged
therealbrad merged 1 commit into
mainfrom
chore/clean-main-from-review-approval
May 19, 2026
Merged

chore: drop accidentally-merged review-approval feature work from main#317
therealbrad merged 1 commit into
mainfrom
chore/clean-main-from-review-approval

Conversation

@therealbrad
Copy link
Copy Markdown
Contributor

Description

Resets main's tree to commit 259e16dc (chore(queues): extract shared defaultJobOptions presets (#316)), the last commit before the accidental review-approval feature merge.

The earlier revert 8a50ad60 only undid the polish wave (9c2fd93b) and the auto-generated 0.28.0 release commit (8a3aac37), but left the underlying review-approval work on main:

  • 01-XX phase commits (ReviewGateError, ReviewRequest schema, etc.)
  • 02-XX phase commits (ReviewInboxButton, RequestReviewSheet, AssigneeCombobox, etc.)
  • WR-XX / CR-XX polish fixes
  • The /reviews route, components/reviews/*, and supporting hooks/services

This PR removes all of that from main in a single revert commit. features/review-approval retains the full history and will land cleanly via its own PR when ready.

Related Issue

Restores main to a clean state after 8a50ad60 (incomplete revert of 9c2fd93b).

Type of Change

  • Other (revert)

Testing

  • git diff --stat HEAD 259e16dc returns empty (tree-equivalent to last clean commit)
  • File count matches 259e16dc (2605 files on both sides)
  • CI green

Checklist

  • Commit is revert: typed (no semantic-release version bump expected)
  • No code changes — pure tree restore to a known-good commit
  • features/review-approval is unaffected (still has full history at 2dffbe5d)

Resets main tree to commit 259e16d, the last commit before the accidental
review-approval merge.

The earlier revert 8a50ad6 only undid the polish wave (9c2fd93) and the
auto-generated 0.28.0 release commit (8a3aac3), leaving the underlying
01-XX and 02-XX phase commits, WR-XX/CR-XX fixes, and the ReviewInboxButton
plus /reviews route on main. This commit removes all of that.

Review-approval feature work continues on features/review-approval and
will land cleanly via PR when ready.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@therealbrad therealbrad merged commit 6eb2245 into main May 19, 2026
5 checks passed
@therealbrad therealbrad deleted the chore/clean-main-from-review-approval branch May 19, 2026 14:01
@therealbrad
Copy link
Copy Markdown
Contributor Author

🎉 This PR is included in version 0.28.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

therealbrad added a commit that referenced this pull request May 19, 2026
…nts (#319)

release-please auto-generated the v0.28.0 section based on commit history
and listed ~30 review-approval features whose source code was reverted in
PR #317 before the release. None of those features actually shipped in
v0.28.0. The GitHub release page has already been corrected; this commit
brings the in-repo CHANGELOG.md in line so future release-please runs
have a clean baseline.

The actual v0.28.0 contents are Turkish/Russian locale support (#318),
the shared queue presets (#316), and the cleanup itself (#317).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant