Skip to content

chore: add PR description template - #1342

Merged
gabrielseco merged 1 commit into
mainfrom
chore/add-pr-template
Sep 21, 2026
Merged

gabrielseco merged 1 commit into
mainfrom
chore/add-pr-template

Conversation

@gabrielseco

@gabrielseco gabrielseco commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Adds a default PR description template so new PRs open with a consistent
shape instead of free-form text.

Why

Diffs show what changed but not why, and that context was getting lost
because nothing prompted for it. Modeled the sections on the GitLab MR
templates used in the dragon/tiger sibling repos, trimmed to what actually
applies to a published npm library rather than a deployed app.

What changed

Toggle details

Added .github/pull_request_template.md with: Summary, Why, What changed
(collapsed), Screenshots (collapsed), Related Resources, Testing.

Deliberately left out, compared to the dragon/tiger source templates:

  • Type of Change section — pr-title.yml already enforces the
    conventional-commit type(scope): prefix on the PR title, so a body
    section would just restate it.
  • Pre-submit checklist — CI covers tests/lint/type-check/exports/size
    fast enough that a manual checklist is noise.
  • PII / backend-branch-pairing checklist items — dragon/tiger-specific
    (Datadog PII redaction, paired backend MR), don't apply to this repo.

Kept a light two-line Testing section (browser test against example/,
feature flag) since this repo does have real feature-flag-gated behavior
(e.g. CurrencyConversionField, bulk GP activation).

Screenshots

N/A — no rendered UI change, this only adds a template file.

Related Resources

N/A

Testing

N/A — template-only change, nothing to run. Verified by opening this PR
itself and confirming the sections above render as intended.


Note

Low Risk
Adds a GitHub markdown template only; no application code, APIs, or CI behavior changes.

Overview
Adds a GitHub pull request template at .github/pull_request_template.md so new PRs open with a fixed outline instead of a blank description.

The template prompts for Summary, Why, and What changed (in a collapsible block), plus optional Screenshots, Related Resources, and a short Testing checklist (browser check against example/, feature flag). Inline HTML comments explain what to write in each section and when to use N/A.

This is process/documentation only; it does not change library code or runtime behavior.

Reviewed by Cursor Bugbot for commit 4d5ea5e. Bugbot is set up for automated code reviews on this repo. Configure here.

Adds .github/pull_request_template.md so PR bodies default to a
consistent shape (Summary, Why, What changed, Screenshots, Related
Resources, Testing) instead of free-form text. Modeled on the
dragon/tiger GitLab MR templates, trimmed to drop sections that don't
apply here: no Type of Change section (pr-title.yml already enforces
the conventional-commit prefix on the title), no pre-submit checklist
(covered by CI), no PII/backend-branch-pairing checklist items.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Deploy preview for adp-cost-calculator ready!

Project:adp-cost-calculator
Status: ✅  Deploy successful!
Preview URL:https://adp-cost-calculator-efbv77yk1-remotecom.vercel.app
Latest Commit:4d5ea5e

Deployed with vercel-action

@github-actions

Copy link
Copy Markdown
Contributor

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-falzknl6t-remotecom.vercel.app
Latest Commit:4d5ea5e

Deployed with vercel-action

@github-actions

Copy link
Copy Markdown
Contributor

📊 Coverage Report

⚪ Coverage unchanged

Metric Current Previous Change Status
Lines 86.19% 86.19% 0% ⚪
Statements 85.77% 85.77% 0% ⚪
Functions 84.73% 84.73% 0% ⚪
Branches 77.66% 77.66% 0% ⚪

Detailed Breakdown

Lines Coverage
  • Covered: 4780 / 5546
  • Coverage: 86.19%
  • Change: 0% (0 lines)
Statements Coverage
  • Covered: 4864 / 5671
  • Coverage: 85.77%
  • Change: 0% (0 statements)
Functions Coverage
  • Covered: 1276 / 1506
  • Coverage: 84.73%
  • Change: 0% (0 functions)
Branches Coverage
  • Covered: 2962 / 3814
  • Coverage: 77.66%
  • Change: 0% (0 branches)

✅ Coverage check passed

@gabrielseco
gabrielseco merged commit 1e3c42e into main Sep 21, 2026
13 checks passed
@gabrielseco
gabrielseco deleted the chore/add-pr-template branch September 21, 2026 08:49
@gabrielseco gabrielseco mentioned this pull request Sep 25, 2026
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