Skip to content

docs: require maintainer approval before merge or deploy#31

Merged
jontsai merged 2 commits into
masterfrom
website-merge-deploy-rules
May 7, 2026
Merged

docs: require maintainer approval before merge or deploy#31
jontsai merged 2 commits into
masterfrom
website-merge-deploy-rules

Conversation

@jontsai

@jontsai jontsai commented May 6, 2026

Copy link
Copy Markdown
Collaborator

Strengthen the public website contributor rules so production changes stop at the PR until the maintainer explicitly approves merge/deploy.

  • Clarifies that implementation/fix requests authorize preparing a PR only, not shipping to production
  • Adds a pre-merge/deploy checklist for explicit approval
  • Documents that rollback authorization is limited to the requested rollback
  • Uses generic maintainer wording appropriate for a public repo

Verification:

  • git diff --check
  • Husky/lint-staged pre-commit completed successfully

Copilot AI review requested due to automatic review settings May 6, 2026 21:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens the project’s operational documentation to ensure merges/deploys to master (production via GitHub Pages) only happen after explicit maintainer approval, and clarifies that implementation/fix requests authorize opening a PR—not shipping changes.

Changes:

  • Clarifies in contributor workflow docs that master is production and requires explicit approval before merge/deploy.
  • Expands agent guidance (“golden rules” + workflow steps) to require stopping at PR until approval is explicitly granted.
  • Adds a pre-merge/deploy checklist (including rollback boundary) across the agent documentation set.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
CONTRIBUTING.md Adds explicit “approval required” language to the contributor workflow and PR checklist.
AGENTS.md Strengthens agent rules and workflow steps to require explicit approval before merge/deploy; adds a pre-merge/deploy checklist.
agent-docs/repository.md Reinforces the deploy model and approval gating in the repository guide.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jontsai jontsai changed the title docs: require explicit approval before merge or deploy docs: require maintainer approval before merge or deploy May 7, 2026
@jontsai jontsai merged commit 7bc7279 into master May 7, 2026
1 check passed
@jontsai jontsai deleted the website-merge-deploy-rules branch May 7, 2026 01:58
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