Skip to content

ci: enforce PR description template#225

Merged
DeadWaveWave merged 1 commit intomainfrom
feat/pr-description-template-check
May 4, 2026
Merged

ci: enforce PR description template#225
DeadWaveWave merged 1 commit intomainfrom
feat/pr-description-template-check

Conversation

@DeadWaveWave
Copy link
Copy Markdown
Owner

💡 Change Scope

  • Small Change: Fast feedback, localized UI/logic, low-risk.
  • Large Change: New feature, cross-boundary logic, runtime-risk (persistence, IPC, lifecycle, recovery).

📝 What Does This PR Do?

  • Adds a lightweight GitHub Actions workflow to validate PR descriptions against .github/pull_request_template.md.
  • Keeps the workflow thin by moving the parsing and validation rules into a reusable Node CLI and library.
  • Adds unit coverage for required sections, exclusive Small/Large selection, mandatory summary content, and required Large Change spec content.
  • Avoids coupling PR metadata validation to the existing heavy CI pipeline.

🏗️ Large Change Spec (Required if "Large Change" is checked)

1. Context & Business Logic

N/A. Small CI/process change only.

2. State Ownership & Invariants

N/A. No runtime state ownership changes.

3. Verification Plan & Regression Layer

N/A. Verified as a Small Change through targeted script tests plus full pnpm pre-commit.


✅ Delivery & Compliance Checklist

  • My code passes the ultimate gatekeeper: pnpm pre-commit is completely green.
  • I have signed the CLA if required (see CLA.md).
  • I have included new tests to lock down the behavior (or explicitly stated why it's untestable).
  • I have strictly adhered to the DEVELOPMENT.md architectural boundaries.
  • I have attached a screenshot or screen recording (if this touches the UI).
  • I have updated the documentation accordingly (if adding a feature or changing a contract).

📸 Screenshots / Visual Evidence

Not applicable. No UI changes.

@DeadWaveWave DeadWaveWave merged commit 69a9ced into main May 4, 2026
8 checks passed
@DeadWaveWave DeadWaveWave deleted the feat/pr-description-template-check branch May 4, 2026 07:27
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