Guidelines too opinionated about squashing commits on develop
@vpanjganj đang làm issue này rồi.
Từ ngày 13/7/2021.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
In 1.2, we can read the following:
You can use --autosquash to squash all your commits to a single commit. Nobody wants many commits for a single feature in develop branch. read more...
I think that asserting that nobody wants that is a fallacy. There are good reasons for which you would want to keep those commits separated even in develop branch:
- know what the reason of a change was when doing a git blame, which is naturally in the commit message/description and not in the feature
- having more precision for git bisect: squashing makes you lose information as you only know which feature caused a regression, rather than a specific commit (the latter narrows the scope)
Those are the reasons why I never squash my commits into a feature commit. You should therefore mention in your guidelines that this is subjective.
- Ngôn ngữ chính
- JavaScript
- Star
- 29.4k
- Fork
- 3.1k
- 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
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 elsewhencode/project-guidelines
-
AQUAEDGE PROMOTIONS Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
elsewhencode/project-guidelines#176 · 1 bình luận ·
-
improvement
elsewhencode/project-guidelines#160 · 3 reaction · 1 người được giao ·
-
improvement
elsewhencode/project-guidelines#140 · 1 người được giao ·
-
Consistent API error messages Đang mởimprovement question
elsewhencode/project-guidelines#134 · 1 người được giao ·
-
suggestion
elsewhencode/project-guidelines#111 · 1 bình luận · 3 reaction · 1 người được giao ·
Tất cả issue của elsewhencode/project-guidelines
Issue tương tự
-
bug confirmed issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
open-webui/open-webui#30750 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Mend: dependency security vulnerability untriaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100