Skip to content

chore: slim down PR description template and update deps#299

Merged
ichoosetoaccept merged 1 commit intomainfrom
chore/slim-pr-template
Apr 12, 2026
Merged

chore: slim down PR description template and update deps#299
ichoosetoaccept merged 1 commit intomainfrom
chore/slim-pr-template

Conversation

@ichoosetoaccept
Copy link
Copy Markdown
Member

Summary

  • Replace verbose PR description template (Why, Notes for reviewer sections) with a minimal Closes # prompt — puts issue linking front and center, removes sections nobody fills in
  • Applied to both template (project/) and repo's own .github/pull_request_template.md
  • Bump dev dependencies via uv sync --upgrade
  • Revert lychee prek rev from nightly back to lychee-v0.23.0 (autoupdate picked up mutable tag)

Closes DOT-330

@ichoosetoaccept
Copy link
Copy Markdown
Member Author

This change is part of the following stack:

Change managed by git-spice.

@linear
Copy link
Copy Markdown

linear bot commented Apr 12, 2026

@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Apr 12, 2026

Greptile Summary

This PR simplifies both the repo's and the scaffold template's PR description template to a minimal Closes # prompt, reverts the lychee prek hook rev from the mutable nightly tag back to lychee-v0.23.0, and bumps several other dev-dependency versions via uv sync --upgrade. All changes are low-risk maintenance: project/prek.toml.jinja already carries the same lychee-v0.23.0 pin (with the matching explanatory comment), so the dual-config invariant described in CLAUDE.md is satisfied.

Confidence Score: 5/5

Safe to merge — purely maintenance changes with no logic or template rendering modifications.

All four changed files are either generated (uv.lock), configuration-only (prek.toml with version bumps and a stable tag revert), or plain markdown templates with no Jinja logic. No code paths are affected, and the dual-config invariant for prek.toml/project/prek.toml.jinja is already satisfied. No P0 or P1 findings.

No files require special attention.

Important Files Changed

Filename Overview
.github/pull_request_template.md PR description template simplified to a minimal "Closes #" prompt with an HTML comment; no issues.
project/.github/pull_request_template.md Template copy of the PR description template simplified identically to the repo's own template; no issues.
prek.toml Lychee rev reverted to stable tag "lychee-v0.23.0" (with explanatory comment), and several hook versions bumped; project/prek.toml.jinja already uses the same lychee rev so no dual-config gap.
uv.lock Generated lock file updated via "uv sync --upgrade"; no manual edits required or expected.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Developer opens PR] --> B["PR template rendered\n(Closes #)"]
    B --> C{Issue linked?}
    C -->|Yes| D[Issue auto-closes on merge]
    C -->|No - delete line| E[PR with no linked issue]
    C -->|Ref - use Ref keyword| F[Issue linked without closing]
Loading

Reviews (1): Last reviewed commit: "chore: slim down PR description template..." | Re-trigger Greptile

@ichoosetoaccept ichoosetoaccept merged commit b6cfbc7 into main Apr 12, 2026
6 checks passed
@ichoosetoaccept ichoosetoaccept deleted the chore/slim-pr-template branch April 12, 2026 14:13
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