Update .github/MAINTAINER.md (#13398)
Co-authored-by: sglang-bot <sglangbot@gmail.com>
This commit is contained in:
@@ -28,12 +28,15 @@ __Note__: Difference between Merge Oncall and Codeowner
|
|||||||
## Pull Request Merge Process
|
## Pull Request Merge Process
|
||||||
1. The author submits a pull request (PR) and fills out the PR checklist.
|
1. The author submits a pull request (PR) and fills out the PR checklist.
|
||||||
2. A bot assigns this PR to a Merge Oncall and @-mentions them. At the same time, GitHub will automatically request reviews from Codeowners.
|
2. A bot assigns this PR to a Merge Oncall and @-mentions them. At the same time, GitHub will automatically request reviews from Codeowners.
|
||||||
3. The Merge Oncall coordinates the review (e.g., asking people to review) and approves the PR; the relevant Codeowners also approve the PR.
|
3. The Merge Oncall coordinates the review (e.g., asking people to review) and approves the PR; the relevant Codeowners also approve the PR. If the assigned Merge Oncall is not responsive, the author can ping other related Merge Oncalls and Reviewers in the list below.
|
||||||
4. The code can now be merged:
|
4. The code can now be merged:
|
||||||
- **Ideal case:** For each modified file, one Codeowner has approved the PR. The PR has also passed the required CI tests. Then, anyone with write permission can merge the PR.
|
- **Ideal case:** For each modified file, one Codeowner has approved the PR. The PR has also passed the required CI tests. Then, anyone with write permission can merge the PR.
|
||||||
- **Exception:** In cases where it is difficult to meet all requirements (due to flaky CI or slow responses), a Merge Oncall can bypass branch protection to merge the PR.
|
- **Exception:** In cases where it is difficult to meet all requirements (due to flaky CI or slow responses), a Merge Oncall can bypass branch protection to merge the PR.
|
||||||
|
|
||||||
|
If you meet any issues during the merge, you can discuss in [slack channels](https://slack.sglang.ai/): #dev, #pull-request, and #ci-cd-build-release.
|
||||||
|
|
||||||
## The List of Merge Oncalls and Reviewers
|
## The List of Merge Oncalls and Reviewers
|
||||||
|
TODO
|
||||||
|
|
||||||
This list is based on the current situation. If you or someone you know would like to take on more responsibility and are qualified, please ping @Lianmin Zheng and @Ying Sheng in the Slack channel. They will start a nomination and internal review process.
|
This list is based on the current situation. If you or someone you know would like to take on more responsibility and are qualified, please ping @Lianmin Zheng and @Ying Sheng in the Slack channel. They will start a nomination and internal review process.
|
||||||
|
|
||||||
@@ -51,3 +54,5 @@ The format is @github-username (Slack username).
|
|||||||
|
|
||||||
### Ascend NPUs
|
### Ascend NPUs
|
||||||
@iforgetmyname (Even Zhou)
|
@iforgetmyname (Even Zhou)
|
||||||
|
|
||||||
|
This list is based on the current situation. If you or someone you know would like to donate machines for CI, they can serve as the CI oncalls for their machines. Please ping @Lianmin Zheng and @Ying Sheng in the Slack channel. They will start a nomination and internal review process.
|
||||||
|
|||||||
@@ -22,3 +22,4 @@
|
|||||||
- [ ] Add unit tests according to the [Run and add unit tests](https://docs.sglang.ai/developer_guide/contribution_guide.html#run-and-add-unit-tests).
|
- [ ] Add unit tests according to the [Run and add unit tests](https://docs.sglang.ai/developer_guide/contribution_guide.html#run-and-add-unit-tests).
|
||||||
- [ ] Update documentation according to [Write documentations](https://docs.sglang.ai/developer_guide/contribution_guide.html#write-documentations).
|
- [ ] Update documentation according to [Write documentations](https://docs.sglang.ai/developer_guide/contribution_guide.html#write-documentations).
|
||||||
- [ ] Provide accuracy and speed benchmark results according to [Test the accuracy](https://docs.sglang.ai/developer_guide/contribution_guide.html#test-the-accuracy) and [Benchmark the speed](https://docs.sglang.ai/developer_guide/contribution_guide.html#benchmark-the-speed).
|
- [ ] Provide accuracy and speed benchmark results according to [Test the accuracy](https://docs.sglang.ai/developer_guide/contribution_guide.html#test-the-accuracy) and [Benchmark the speed](https://docs.sglang.ai/developer_guide/contribution_guide.html#benchmark-the-speed).
|
||||||
|
- [ ] Work with maintainers to merge your PR. See the [PR Merge Process](https://github.com/sgl-project/sglang/blob/main/.github/MAINTAINER.md#pull-request-merge-process)
|
||||||
|
|||||||
@@ -63,10 +63,10 @@ You can find additional accuracy eval examples in:
|
|||||||
## Benchmark the speed
|
## Benchmark the speed
|
||||||
Refer to [Benchmark and Profiling](../developer_guide/benchmark_and_profiling.md).
|
Refer to [Benchmark and Profiling](../developer_guide/benchmark_and_profiling.md).
|
||||||
|
|
||||||
## Request a review
|
## Requesting a review for merge
|
||||||
You can identify potential reviewers for your code by checking the [code owners](https://github.com/sgl-project/sglang/blob/main/.github/CODEOWNERS) and [reviewers](https://github.com/sgl-project/sglang/blob/main/.github/REVIEWERS.md) files.
|
You can follow the pull request merge process described in [MAINTAINER.md](https://github.com/sgl-project/sglang/blob/main/.github/MAINTAINER.md).
|
||||||
Another effective strategy is to review the file modification history and contact individuals who have frequently edited the files.
|
You will need to work with the Merge Oncall, Codeowner, and other reviewers to get their approvals.
|
||||||
If you modify files protected by code owners, their approval is required to merge the code.
|
Then your PR can be merged.
|
||||||
|
|
||||||
## How to trigger CI
|
## How to trigger CI
|
||||||
To trigger CI, the pull request must have the "run-ci" label.
|
To trigger CI, the pull request must have the "run-ci" label.
|
||||||
|
|||||||
Reference in New Issue
Block a user