Skip to content

Commit

Permalink
Merge pull request #1216 from 1C-Company/iArtemv-patch-1
Browse files Browse the repository at this point in the history
Приведено в соответствие с описанием, в конце описания комита точка
  • Loading branch information
marmyshev committed Nov 29, 2022
2 parents 6d56922 + 514e955 commit 72afdbb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/contributing/commits.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
```
#<НОМЕР ЗАДАЧИ> Короткое сообщение что сделано
Длинное сообщение что сделано
Длинное сообщение что сделано.
```

1. Начинаем с номера задачи (например, #12311). Идентификатор необходим для интеграции с задачами GitHub
Expand Down Expand Up @@ -49,4 +49,4 @@
Стоит ли делать один коммит на фичу - зависит от фичи. Коммит - логически завершенное изменение, а сложная фича может содержать много таких изменений, слияние которых в один коммит, приводит к потере информации о деталях ее выполнения. Это касается не только фич.

Squash'ить коммиты стоит, если были сделаны лишнее изменения, которые в следующих коммитах отменили. Так же, стоит избегать коммитов вида "Поднятие версий", которые не являются самостоятельными (вызваны другими изменениями, без них билд проекта не будет выполнен).


0 comments on commit 72afdbb

Please sign in to comment.