From 83335c2f39458f2f3fd19029960bd1ec31de031e Mon Sep 17 00:00:00 2001 From: mateo-berri <277851410+mateo-berri@users.noreply.github.com> Date: Sat, 15 Aug 2026 16:37:21 -0700 Subject: [PATCH] docs(github): proof-of-fix template shows only the latest run as Before/After with nested cases --- .github/pull_request_template.md | 34 +++++++++++++++++++++++++++----- 1 file changed, 29 insertions(+), 5 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index e56f61988ef..c9fd9d84766 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -65,11 +65,35 @@ If you're seeing a delay in your PR being merged, ping the LiteLLM Team on [Slac + Show ONLY the latest run: capture Before at the merge base and After at the PR's current tip, and when new commits change behavior, replace this whole section with the fresh run instead of stacking it on top of older ones + Structure the section exactly as below: Before and After one heading level below this section, each naming the commit hash it was captured at, one lower-level heading per case inside each, the same case names in the same order on both sides; shared setup (config, payloads) goes above Before, and with a single case, drop the case headings and number the steps directly + +### Before () + +#### + +1. ... +2. ... + +#### + +1. ... + +### After () + +#### + +1. ... +2. ... + +#### + +1. ... + + For bug fixes: Before shows the reproduction, After shows the same steps passing + For new features: Before shows the capability missing, After shows it working end-to-end + If the change applies to all three LLM endpoints (/v1/responses, /v1/chat/completions, /v1/messages), make each endpoint its own case, not just one + For UI changes: before/after screenshots under the same headings --> ## Type