Skip to content

Add Pull Request reviews to Push MD#77

Merged
artpi merged 1 commit into
trunkfrom
codex/push-md-pull-requests
Jul 14, 2026
Merged

Add Pull Request reviews to Push MD#77
artpi merged 1 commit into
trunkfrom
codex/push-md-pull-requests

Conversation

@artpi

@artpi artpi commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • store preview branches as a private push_md_pull_request CPT with native lifecycle statuses
  • add WordPress Notes for general and inline diff comments
  • add comment-driven, extensible PR review states
  • add editable PR descriptions and commit history with commit message bodies
  • add a GitHub-like review UI with previews, diffs, lifecycle groups, and merge actions
  • migrate legacy branch preview options and remove PR data on uninstall

REST and permissions

  • use the native post and comment REST controllers
  • expose calculated changed files, line diffs, preview URLs, and commit history as read-only fields
  • allow admins to update only the description of active PRs
  • keep lifecycle, protected metadata, merged/closed PRs, and state-changing Notes guarded

Testing

  • vendor/bin/phpcs plugins/push-md
  • vendor/bin/phpunit plugins/push-md/Tests/
  • full WordPress E2E: 31 tests, 698 assertions
  • node --check plugins/push-md/admin-shell.js
  • git diff --check

@artpi artpi self-assigned this Jul 14, 2026
@artpi
artpi merged commit fb0042c into trunk Jul 14, 2026
44 of 45 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