|
| 1 | +## Description |
| 2 | + |
| 3 | +<!-- Provide a clear and concise description of what this PR does --> |
| 4 | + |
| 5 | +## Motivation |
| 6 | + |
| 7 | +<!-- Why is this change needed? Link to related issues if applicable --> |
| 8 | + |
| 9 | +Closes #<!-- issue number --> |
| 10 | + |
| 11 | +## Type of Change |
| 12 | + |
| 13 | +<!-- Mark the relevant option with an "x" --> |
| 14 | + |
| 15 | +- [ ] Bug fix (non-breaking change that fixes an issue) |
| 16 | +- [ ] New feature (non-breaking change that adds functionality) |
| 17 | +- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) |
| 18 | +- [ ] Documentation update |
| 19 | +- [ ] Code refactoring (no functional changes) |
| 20 | +- [ ] Performance improvement |
| 21 | +- [ ] Test coverage improvement |
| 22 | +- [ ] CI/CD or tooling change |
| 23 | + |
| 24 | +## Changes Made |
| 25 | + |
| 26 | +<!-- List the main changes made in this PR --> |
| 27 | + |
| 28 | +- |
| 29 | +- |
| 30 | +- |
| 31 | + |
| 32 | +## Testing |
| 33 | + |
| 34 | +<!-- Describe the tests you ran and how to reproduce them --> |
| 35 | + |
| 36 | +### Test Coverage |
| 37 | + |
| 38 | +- [ ] Unit tests added/updated |
| 39 | +- [ ] Integration tests added/updated (if applicable) |
| 40 | +- [ ] All existing tests pass locally |
| 41 | + |
| 42 | +### Manual Testing |
| 43 | + |
| 44 | +<!-- Describe any manual testing performed --> |
| 45 | + |
| 46 | +#### Method Used |
| 47 | + |
| 48 | +- [ ] OpenAI Python SDK |
| 49 | +- [ ] curl |
| 50 | +- [ ] Docker |
| 51 | +- [ ] Other: <!-- specify --> |
| 52 | + |
| 53 | +<details> |
| 54 | +<summary>Test commands / code</summary> |
| 55 | + |
| 56 | +**Example with OpenAI SDK:** |
| 57 | + |
| 58 | +```python |
| 59 | +from openai import OpenAI |
| 60 | + |
| 61 | +client = OpenAI(base_url="http://localhost:8090", api_key="your-key") |
| 62 | + |
| 63 | +completion = client.chat.completions.create( |
| 64 | + model="gpt-4", |
| 65 | + messages=[ |
| 66 | + {"role": "user", "content": "Test message"} |
| 67 | + ], |
| 68 | +) |
| 69 | +print(completion.choices[0].message.content) |
| 70 | +``` |
| 71 | + |
| 72 | +**Example with curl:** |
| 73 | + |
| 74 | +```bash |
| 75 | +curl -X POST http://localhost:8090/v1/chat/completions \ |
| 76 | + -H "Content-Type: application/json" \ |
| 77 | + -H "Authorization: Bearer your-key" \ |
| 78 | + -d '{ |
| 79 | + "model": "gpt-4", |
| 80 | + "messages": [{"role": "user", "content": "Test"}] |
| 81 | + }' |
| 82 | +``` |
| 83 | + |
| 84 | +</details> |
| 85 | + |
| 86 | +## Checklist |
| 87 | + |
| 88 | +<!-- Mark completed items with an "x" --> |
| 89 | + |
| 90 | +### Code Quality |
| 91 | + |
| 92 | +- [ ] Code follows the project's style guidelines |
| 93 | +- [ ] I have performed a self-review of my code |
| 94 | +- [ ] I have commented my code, particularly in hard-to-understand areas |
| 95 | +- [ ] My changes generate no new linter warnings (`make lint`) |
| 96 | +- [ ] Type checking passes (`make mypy`) |
| 97 | +- [ ] All tests pass (`make test`) |
| 98 | + |
| 99 | +### Documentation |
| 100 | + |
| 101 | +- [ ] I have updated the documentation accordingly |
| 102 | +- [ ] Docstrings follow Google style with imperative mood |
| 103 | +- [ ] I have added examples for new features (if applicable) |
| 104 | +- [ ] README.md updated (if applicable) |
| 105 | + |
| 106 | +### Dependencies |
| 107 | + |
| 108 | +- [ ] No new dependencies added |
| 109 | +- [ ] If dependencies added, they are justified and minimal |
| 110 | +- [ ] `uv.lock` updated (if dependencies changed) |
| 111 | + |
| 112 | +### Compatibility |
| 113 | + |
| 114 | +- [ ] Changes are compatible with Python 3.10-3.14 |
| 115 | +- [ ] No use of `type[...]` syntax (use `Type[...]` for Python 3.8) |
| 116 | +- [ ] Async/sync variants both work correctly (if applicable) |
| 117 | + |
| 118 | +### Commits |
| 119 | + |
| 120 | +- [ ] Commit messages are clear and follow conventional commits style |
| 121 | +- [ ] Commits are logically organized |
| 122 | +- [ ] No debug code or commented-out code left in |
| 123 | + |
| 124 | +## Additional Context |
| 125 | + |
| 126 | +<!-- Add any other context, screenshots, or information about the PR here --> |
| 127 | + |
| 128 | +## Pre-merge Actions |
| 129 | + |
| 130 | +<!-- For maintainers --> |
| 131 | + |
| 132 | +- [ ] Changelog updated (if applicable) |
| 133 | +- [ ] Version bump considered (if applicable) |
0 commit comments