search

tlbootcamp / tlroadmap

Тимлид – это ❄️, потому что в каждой компании он уникален и неповторим.
https://tlroadmap.io
Creative Commons Attribution Share Alike 4.0 International
5.05k stars 491 forks source link

Организовать CI #112

Closed etolstoy closed 5 years ago

etolstoy commented 5 years ago

Довольно быстро накопились разные проверки и скрипты, которые нужно дергать при каждом мердже:

Было бы круто наладить нормальный CI, чтобы оно вот само работало.

php-coder commented 5 years ago

А если к этому всему еще и https://danger.systems/js/ прикрутить...

timmson commented 5 years ago

Попробовал yaspeller в Travis CI - https://travis-ci.org/timmson/tlroadmap/jobs/576285307

Как думаете? Стоит прикручивать?

etolstoy commented 5 years ago

Думаю да – уже пролезают PR с ошибками. Туда же и генерацию README.

timmson commented 5 years ago

Кину следующий PR со спеллером, а потом докручу генератор. Обрытный пуш после генерации можно так положить - https://gist.github.com/willprice/e07efd73fb7f13f917ea, а ключик просписать в переменных окружения билд плана, чтобы был доступен только авторам репозитория

etolstoy commented 5 years ago

@timmson еще стоит добавить логику отсюда - https://github.com/tlbootcamp/tlroadmap/pull/111

timmson commented 5 years ago

@timmson еще стоит добавить логику отсюда - #111

Если я все правильно сделал, то получается как-то так - https://travis-ci.org/timmson/tlroadmap/builds/578031320

etolstoy commented 5 years ago

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

DevAlloy commented 5 years ago

Еще надо бы запускать генерацию ссылок из links.json и проверять, что автор PR уже сам прогнал скрипт.

vvscode commented 5 years ago

@timmson сделаешь PR или этот таск свободен?

timmson commented 5 years ago

@vvscode, забирай. Я уже накидал конфигу для tavis ci, можно ее взять за основу - https://github.com/timmson/tlroadmap/blob/ci/.travis.yml

vvscode commented 5 years ago

@etolstoy как на счет добавления package.json в проект?

1) там можно описать команды для проверки репо 2) там можно описать lint-staged хуки для проверки перед коммитом

vvscode commented 5 years ago

Плюс - мы добавляем линтинг, нам бы в секции README.md или в файле CONTRIBUTION.md добавить описание этих проверок

мне это в том же PR сделать? или в отдельном?

vvscode commented 5 years ago

И в догонку - в python используется lxml, но никакого requirement.txt я не вижу ))

Как на счет переписать все на js? ;)

etolstoy commented 5 years ago

@vvscode eyo странно работает, предлагает заменять "чем конкретно хотите" на "чём конкретно хотите". Как сделать его менее геморройным, чем вручную настраивать исключения для таких случаев? :)

etolstoy commented 5 years ago

Вообще я за то, чтобы использовать один язык в репозитории. На питоне сейчас написан довольно большой кусок логики по синхронизации базы знаний и майндмепа. Что мы получим от переписывания его на js?

vvscode commented 5 years ago

@etolstoy
1) про eyo - есть флаг --only-safe 2) про js - вы никаких бонусов не получите )) я просто уточнил, потому что ci сейчас на js настроен, а к python коду нет файла зависимостей - это я поправлю. Просто уточнил, что будем использовать - разницы для меня нет

etolstoy commented 5 years ago

Кажется, подняли.