You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
### What this PR does / why we need it?
Add reminder comment when PR submitted
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Test locally:
Yikun#51 (comment)
This PR will take effect after this PR merged.
- vLLM version: v0.10.0
- vLLM main:
vllm-project/vllm@0e36abf
Signed-off-by: Yikun Jiang <[email protected]>
body: '👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:\n\n' +
20
+
'- A PR should do only one thing, smaller PRs enable faster reviews.\n' +
21
+
'- Every PR should include unit tests and end-to-end tests to ensure it works and is not broken by other future PRs.\n' +
22
+
'- Write the commit message by fulfilling the PR description to help reviewer and future developers understand.\n\n' +
23
+
'If CI fails, you can run linting and testing checks locally according [Contributing](https://vllm-ascend.readthedocs.io/zh-cn/latest/developer_guide/contribution/index.html) and [Testing](https://vllm-ascend.readthedocs.io/zh-cn/latest/developer_guide/contribution/testing.html).'
0 commit comments