Skip to content

Добавлены правила оформления коммитов#2

Open
magauran wants to merge 1 commit intomasterfrom
commits
Open

Добавлены правила оформления коммитов#2
magauran wants to merge 1 commit intomasterfrom
commits

Conversation

@magauran
Copy link
Copy Markdown
Contributor

No description provided.

@magauran
Copy link
Copy Markdown
Contributor Author

ping @anivaros, @korDmitry

Comment thread README.md
- Создавая коммит, подумайте, какая информация понадобится вам, если вы наткнётесь на этот коммит через год.

Структура Message коммита:
> [Ключ задачи в JIRA] Краткое описание изменений.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 - зачем квадратные скобки?
2 - я все таки предлагаю писать не только код, но и название задачи, проще ориентироваться в коммитах будет, не надо каждый раз в джиру лезть

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Чтобы как-то разделять от описания.
  2. Если добавлять ещё и название задачи, то ничего содержательного уже не поместится. И повторится та же проблема, что и сейчас - по Subject коммита не понятно, о чём он.

Может вообще название задачи в Description вынести?

Copy link
Copy Markdown
Contributor

@anivaros anivaros May 24, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в Description

Это где?

Comment thread README.md

## Оформление коммитов
- Коммит должен содержать только одно логическое изменение.
- Первая строка коммита (Subject) должна содержать не более 72 символов.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Никто не будет считать символы)

Comment thread README.md
## Структура проекта


## Оформление коммитов
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я для арма писал такие правила:

Работа с коммитами
Каждый коммит должен содержать в себе логически законченное изменение. В него не должны попадать какие либо другие изменения.

Комментарий к коммиту должен быть написан на русском языке, содержать ID и заголовок задачи и краткое описание изменения.

Пример комментария:

EW_IOS-98 требуется собрать приложение с использованием iBM Maas 360 SDK 2.95

Обновил структуру и настройки проекта

Обновил фреймворк MaaS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants