Skip to content

chore: standardize PR descriptions on functional change + before/after - #33

Merged
chaliy merged 1 commit into
mainfrom
claude/pr-description-standards-0xh7do
Jul 8, 2026
Merged

chore: standardize PR descriptions on functional change + before/after#33
chaliy merged 1 commit into
mainfrom
claude/pr-description-standards-0xh7do

Conversation

@chaliy

@chaliy chaliy commented Jul 8, 2026

Copy link
Copy Markdown
Owner

What changed

PR descriptions now center on functional change and impact instead of a code-location walkthrough, and the repo gains a .github/pull_request_template.md with a Before / After section that asks for proof — CLI output, logs, or screenshots for UI. The inline PR-body template in AGENTS.md is updated to match.

Why

The diff already shows where and how code changed. A description should add what the diff can't: the behavior that changed, who it affects, and evidence it works — shifting effort from restating code to demonstrating impact.

Before / After

Docs/templates only — no runtime behavior changes.

  • Before: no PR template; inline body used ## What / ## How.
  • After: ## What changed (functional summary + impact) + ## Before / After (proof: output/logs, screenshots for UI when possible); template file now exists and matches AGENTS.md.

Risk

  • Low
  • Documentation and PR-template only; no source, config, or CI logic touched.

Checklist

  • Unit tests are passed — N/A (docs/templates only)
  • Smoke tests are passed — N/A
  • Documentation is updated

Generated by Claude Code

@chaliy
chaliy merged commit cd4405d into main Jul 8, 2026
1 check passed
@chaliy
chaliy deleted the claude/pr-description-standards-0xh7do branch July 8, 2026 20:02
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