mirror of
https://github.com/tiennm99/litellm.git
synced 2026-06-17 16:48:54 +00:00
Fix: simplify PR template
This commit is contained in:
@@ -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
|
||||
|
||||
|
||||
|
||||
Reference in New Issue
Block a user