Skip to content

Docs: update pull request template#62

Merged
SilverSupplier merged 1 commit into
sw1029:mainfrom
SilverSupplier:codex/pr-template-validation-notes
May 17, 2026
Merged

Docs: update pull request template#62
SilverSupplier merged 1 commit into
sw1029:mainfrom
SilverSupplier:codex/pr-template-validation-notes

Conversation

@SilverSupplier
Copy link
Copy Markdown
Collaborator

Summary

  • Update the repository pull request template to match the review format used by recent PRs.
  • Add Unity EditMode/PlayMode and GitHub checks to the default verification checklist.
  • Add scope, out-of-scope, risk, screenshots/playtest, and issue-link sections so future PR bodies keep review context visible.

Verification

  • node scripts/validate-doc-state.mjs
  • git diff --check
  • npm test not run; template-only change.
  • npm run build not run; template-only change.
  • Unity EditMode tests not run; template-only change.
  • Unity PlayMode tests not run; template-only change.

Notes

  • Scope: PR template only.
  • Out of scope / follow-up: existing open PR bodies were already normalized separately.
  • Risk notes: low; this only affects default text shown when opening future PRs.

Screenshots / Playtest Notes

Not applicable; no UI/gameplay change.

@SilverSupplier SilverSupplier requested a review from sw1029 as a code owner May 17, 2026 15:33
@SilverSupplier SilverSupplier merged commit fbf74c2 into sw1029:main May 17, 2026
5 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