From 41b67b577b1b65bf8c32bcdfd1b40095b93a16b5 Mon Sep 17 00:00:00 2001 From: AlexsanderHamir Date: Mon, 15 Dec 2025 07:44:47 -0800 Subject: [PATCH] Fix: simplify PR template --- .github/pull_request_template.md | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index b0a5169cd1..8977332ee0 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -11,17 +11,12 @@ **Please complete all items before asking a LiteLLM maintainer to review your PR** - [ ] I have Added testing in the [`tests/litellm/`](https://github.com/BerriAI/litellm/tree/main/tests/litellm) directory, **Adding at least 1 test is a hard requirement** - [see details](https://docs.litellm.ai/docs/extras/contributing_code) -- [ ] I have added a screenshot of my new test passing locally +- [ ] I have added a screenshot of my new test passing locally - [ ] My PR passes all unit tests on [`make test-unit`](https://docs.litellm.ai/docs/extras/contributing_code) - [ ] My PR's scope is as isolated as possible, it only solves 1 specific problem - ## CI (LiteLLM team) -> 1. Don't merge if CI has insufficient funds issues; ping @AlexsanderHamir until they’re resolved. -> 2. Split your work into small, logical commits and push each one to trigger CI, making it easier **for you** to identify where something broke by comparing against your branch-creation and previous CI runs. -> 3. When merging your PR, choose `Merge & Squash` for convenience. -> > **CI status guideline:** > > - 50-55 passing tests: main is stable with minor issues. @@ -34,7 +29,7 @@ - [ ] **CI run for the last commit** Link: -- [ ] **Merges / cherry-picks into this branch** +- [ ] **Merge / cherry-pick CI run** Links: ## Type @@ -50,5 +45,3 @@ ✅ Test ## Changes - -