Skip to content

chore: make pr template linear portion clearer - #30766

Merged
mateo-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_pr_template_linear_clarity
Jun 19, 2026
Merged

chore: make pr template linear portion clearer#30766
mateo-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_pr_template_linear_clarity

Conversation

@mateo-berri

@mateo-berri mateo-berri commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Relevant issues

Linear ticket

Pre-Submission checklist

Docs-only change to the PR template, so no unit tests apply

Why

This is a copy of #30750 on a litellm_-prefixed branch so the required CircleCI checks run; CircleCI only triggers on litellm_-prefixed branches, so the original mateo-berri-patch-1 branch could not be merged because that required check never ran

Reasoning for the change itself: an internal contributor wrote "No need. External contributor" in the Linear section, so their agent didn't have enough context to know the instruction applied to them. This clarifies that internal contributors are typically those whose GitHub username is postfixed with -berri or -berriai, and spells out the "Resolves " prefix needed for the magic link

Screenshots / Proof of Fix

Docs-only change to .github/pull_request_template.md. The Linear ticket comment now reads:

<!-- if you are an internal contributor (e.g., your username is postfixed with -berri or -berriai), add "Resolves " followed by the Linear ticket e.g. "Resolves LIT-1234" to magically link the Linear ticket to the GitHub PR -->

Type

📖 Documentation

Changes

Clarifies the Linear ticket section of the PR template so internal contributors recognize the instruction applies to them and know to prefix the ticket with "Resolves "

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mateo-berri
mateo-berri marked this pull request as ready for review June 19, 2026 02:06
@greptile-apps

greptile-apps Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the PR template's Linear ticket comment to clarify that "internal contributor" means someone whose GitHub username is postfixed with -berri or -berriai, and to make explicit that the ticket entry must start with "Resolves " to trigger the automatic GitHub↔Linear link.

  • The change is limited to a single HTML comment in .github/pull_request_template.md — no production code is affected.
  • Motivation is to prevent internal contributors (or their agents) from skipping the Linear ticket entry because the original wording was ambiguous about who the instruction applied to.

Confidence Score: 5/5

This is a docs-only change to a single HTML comment in the PR template; it has no impact on any runtime behavior.

Only one line is touched — a comment inside the PR template — and the wording is accurate and unambiguous. No code paths, tests, or configs are affected.

No files require special attention.

Important Files Changed

Filename Overview
.github/pull_request_template.md Single-line comment update clarifying who qualifies as an internal contributor and spelling out the required "Resolves " prefix for Linear ticket linking.

Reviews (1): Last reviewed commit: "chore: make pr template linear portion c..." | Re-trigger Greptile

@mateo-berri
mateo-berri merged commit 31eca17 into litellm_internal_staging Jun 19, 2026
123 checks passed
@mateo-berri
mateo-berri deleted the litellm_pr_template_linear_clarity branch June 19, 2026 06:29
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 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