Conversation
analsituation
requested review from
DavidGuryanov,
andreitakarski,
evgen-kent,
jjswifty,
puptup,
sashabull66 and
vahrammer
as code owners
analsituation
force-pushed
the
feat/DANISJS-6
branch
from
38e9af1 to
95b9291
Compare
vahrammer
left a comment
vahrammer
left a comment
There was a problem hiding this comment.
Привет, в целом все хорошо. По оформлению пару комментов оставил.
| gmt_time=$(TZ="GMT+3" date +%d-%m-%Y); | ||
|
|
||
| echo "$caption $VERSION ([$SHA_SHORT]($LAST_COMMIT_URL)) ($gmt_time)" | ||
| echo "## $VERSION ([$SHA_SHORT]($LAST_COMMIT_URL)) ($gmt_time)" |
There was a problem hiding this comment.
note: В CHANGELOG.md заголовок второго уровня используется только для Minor-заголовков.
Предлагаю либо изменить в текущем CHANGELOG.md все заголовки на второй уровень, либо тут подставлять заголовок 1-3 уровня в зависимости от типа версии (Major/Minor/Patch).
There was a problem hiding this comment.
Я посмотрел и предложил бы ставить заголовок первого уровня всегда. И в README.md добавил бы требование — в описании изменений не использовать заголовки 1-го уровня.
README.md
Outdated
| Для выпуска новой версии библиотеки в npm необходимо создать GitHub Release. При этом нужно: | ||
|
|
||
| 1. Указать тег, который отражает тип изменения: major, minor или patch. | ||
| 2. Заполнить changelog в формате Markdown, описав изменения в разделах: FEATURES, BUGFIXES, BREAKING_CHANGES. |
There was a problem hiding this comment.
suggesion: Давай дополним, что использовать заголовки первого уровня в описании запрещено, т.к. первый уровень используется для основного заголовка версии.
3 participants
Описание
Изменен процесс выпуска новых версий библиотеки. Новый процесс описан в Readme.