Define a pull request template
@benoit74 đang làm issue này rồi.
Từ ngày 22/12/2023.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Some projects have a pull_request_template.yml, but it does not look like there is a convention on this yet.
The template I've found so far is:
## Rationale
[//]: # (Briefly explain the reason behind this change.)
<!--
Issue: [Title](link) or #123 for Github issues.
-->
## Changes
[//]: # (Summarize what has changed.)
Some inconvenient I find in this template:
- I do not find the term "Rationale" very obvious
- There is no mention about the fact that we must add "Fix: ###" to automatically close issues.
- The "Issue" template is misleading, it looks like it will do something automatically while it doesn't + all issues should be Github issues (or most at least), so no need to propose something for the case where it is not
ChatGPT suggested me the following sections:
## Description
<!-- Describe the purpose of this pull request -->
## Changes Made
<!-- List the changes made in this pull request -->
## Screenshots (if applicable)
<!-- Include screenshots or images demonstrating the changes -->
## Testing
<!-- Explain how the changes were tested -->
## Related Issues
<!-- Mention any related issues or link to them -->
## Checklist
- [ ] Code follows project standards
- [ ] Tests have been added/updated
- [ ] Documentation has been updated
- [ ] Changelog entry added
And it should be named pull_request_template.md (it's not a yaml file 😅)
My remarks on ChatGPT suggestion:
- I think the related issues should be clearer about the fact that we expect one or more "Fix: ###" and I would like to put this at the beginning of the PR, just after the description (or even before that) so that it is more obvious why we are proposing this PR
- in the checklist:
- we should add a link to our project standards so that it is obvious
- most scrappers do not have tests (yet, at least) ; and usually there structure makes testing a daunting task ; I think we should not add this check in most scrapers yet
- documentation should refer to usage + wiki
I welcome all suggestions on this, I can handle the creation of a PR once arguments have settled a little bit.
- Ngôn ngữ chính
- Python
- Star
- 1
- Fork
- 2
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của openzim/_python-bootstrap
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 76/100
openzim/_python-bootstrap#57 ·
-
enhancement
openzim/_python-bootstrap#54 · 1 bình luận · 2 người được giao ·
-
Workflow convention Đang mởquestion
openzim/_python-bootstrap#53 · 1 người được giao ·
-
Move `.pre-commit.yaml` to hatch Đang mởenhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
openzim/_python-bootstrap#51 · 3 bình luận ·
-
What about nested logs Đang mởenhancement
openzim/_python-bootstrap#48 · 2 người được giao ·
Tất cả issue của openzim/_python-bootstrap
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
anthropics/skills#1811 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
speaches-ai/speaches#678 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
datalayer/mcp-compose#42 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
conda-forge/spacy-feedstock#177 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
UKGovernmentBEIS/inspect_evals#2523 ·