Skip to content

Smooth release publish handoff#42

Merged
eric-tramel merged 1 commit intomainfrom
codex/release-low-touch-cleanup
May 8, 2026
Merged

Smooth release publish handoff#42
eric-tramel merged 1 commit intomainfrom
codex/release-low-touch-cleanup

Conversation

@eric-tramel
Copy link
Copy Markdown
Contributor

What

Add an explicit PUBLISH=1 mode to make release so maintainers can run local release checks, create or reuse the plugin release tag, push the tag, and publish the GitHub Release in one command.

Also remove stale release-owner plumbing and documentation that still described the old email/CODEOWNERS gate.

Why

After the release workflow moved to GitHub Release events and Pages deployment moved behind the Documentation workflow, the remaining manual handoff was copying the generated tag into git push and gh release create. This keeps the safety checks while making the common authorized release path lower-touch.

Usage

Manual inspect-then-publish path still works:

make release PLUGIN=data-designer-my-plugin
git push origin data-designer-my-plugin/v0.2.0
gh release create data-designer-my-plugin/v0.2.0 \
  --title "data-designer-my-plugin v0.2.0" \
  --notes "Release data-designer-my-plugin v0.2.0" \
  --latest=false \
  --verify-tag

Lowest-touch authorized publish path:

make release PLUGIN=data-designer-my-plugin PUBLISH=1

How

  • Add check-release-state so make release requires a clean worktree and a commit reachable from origin/main before tagging.
  • Make release tag creation idempotent when the existing local tag already points at HEAD.
  • When PUBLISH=1, push the tag and create the GitHub Release with --verify-tag.
  • Delete the unused ddp.release_owners helper left over from the removed owner gate.
  • Update README.md, docs/releasing.md, and AGENTS.md to match the current release contract.

Validation

  • make -n release PLUGIN=data-designer-template
  • make -n release PLUGIN=data-designer-template PUBLISH=1
  • git diff --check
  • make sync
  • make check
  • make docs
  • make lint
  • make test-devtools

@eric-tramel eric-tramel marked this pull request as ready for review May 8, 2026 19:37
@eric-tramel eric-tramel requested a review from a team as a code owner May 8, 2026 19:37
@eric-tramel eric-tramel merged commit f1b3a6a into main May 8, 2026
6 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