Commitlint + Husky работают отлично
Пришлось вручную устанавливать husky, что на практике достаточно плохо. Для того, чтобы этого избежать достаточно добавить в package.json скрипт "prepare: husky install". При "npm ci" эта команда выполнится автоматически, что упрощает жизнь разработчикам)
PR проверки запускаются, но нет запрета на merge при не пройденных проверках
Это можно сделать через настройки репозитория
Релизный issue создается, но не закрывается по окончанию CI. Тут я учитываю вариант, что деплой можно делать вручную, как у тебя это реализовано сейчас и потом закрывать самому Issue. Поэтому здесь все хорошо.
Ссылок или содержимого артифактов нет в Release Issue.
Здесь я вижу несколько вариантов реализации. Ты сделал реализации по отдельности и это может создавать определенные проблемы, когда мы хотим дополнять наш Release Issue, потому что не можем напрямую гарантировать, что CI по созданию Release успешно завершился.
Лучше всего сделать 1 CI, где все будет происходить и в нем у разных этапов будут последовательные зависимости от прошлых этапов. Таким образом мы гарантируем определенные шаги и между этапами можем передавать данные(те же артифакты). Например, на 1 этапе создали открытый Release Issue, далее делаем проверку тестами и обновляем Issue, делаем deploy и обновляем Issue, если все эти этапы пройдены успешно, то можем закрыть Issue.
Деплой не работает
Моя общая оценка: 4/6
Надеюсь, что мой ответ поможет писать хорошие пайплайны. Я написал свое виденье на выполнение этого задания, если заметишь, что неправильно понял тебя, то жду в Telegram)
denisso
commented
1 year ago
Commitlint + Husky работают отлично Пришлось вручную устанавливать husky, что на практике достаточно плохо. Для того, чтобы этого избежать достаточно добавить в package.json скрипт "prepare: husky install". При "npm ci" эта команда выполнится автоматически, что упрощает жизнь разработчикам)
PR проверки запускаются, но нет запрета на merge при не пройденных проверках Это можно сделать через настройки репозитория
Релизный issue создается, но не закрывается по окончанию CI. Тут я учитываю вариант, что деплой можно делать вручную, как у тебя это реализовано сейчас и потом закрывать самому Issue. Поэтому здесь все хорошо.
Ссылок или содержимого артифактов нет в Release Issue.
Здесь я вижу несколько вариантов реализации. Ты сделал реализации по отдельности и это может создавать определенные проблемы, когда мы хотим дополнять наш Release Issue, потому что не можем напрямую гарантировать, что CI по созданию Release успешно завершился.
Лучше всего сделать 1 CI, где все будет происходить и в нем у разных этапов будут последовательные зависимости от прошлых этапов. Таким образом мы гарантируем определенные шаги и между этапами можем передавать данные(те же артифакты). Например, на 1 этапе создали открытый Release Issue, далее делаем проверку тестами и обновляем Issue, делаем deploy и обновляем Issue, если все эти этапы пройдены успешно, то можем закрыть Issue.
Моя общая оценка: 4/6 Надеюсь, что мой ответ поможет писать хорошие пайплайны. Я написал свое виденье на выполнение этого задания, если заметишь, что неправильно понял тебя, то жду в Telegram)