hexlet-interview icon indicating copy to clipboard operation
hexlet-interview copied to clipboard

Добавить pre-commit

Open grigori-gru opened this issue 6 years ago • 5 comments

Можно создать pre-commit и добавить в него:

  • форматирование (запустил npm run format, у меня в 20 файлах изменения)
  • линтер
  • тесты

Хорошая таска для новичков

grigori-gru avatar Sep 24 '19 10:09 grigori-gru

Тут да, есть такое. Единственное мне не нравится husky для этого - потому что он кучу шума создает у меня в папку с гитом.

jougene avatar Sep 24 '19 12:09 jougene

По поводу формата еще. При ci выполняется команда make lint, который использует tslint. format же использует prettier и получается два различных набора правил для просто линта и для линта с фиксом. Поэтому щас есть lint и lint:fix. А как срастить и приеттиер и тслинт - жду предложений.

jougene avatar Sep 24 '19 12:09 jougene

Мне тоже husky не нравится, юзаю эту либу. Правда форматирование не попадет в коммит с ним.

grigori-gru avatar Sep 24 '19 12:09 grigori-gru

А какие проблемы между prettier и линтером? У меня они вместе нормально работают, я Format on Save в vscode ставлю и он сразу в нужном виде сохраняется после редактирования

grigori-gru avatar Sep 24 '19 12:09 grigori-gru

Предлагаю в прекоммите запускать только линтер, а фиксер, при необходимости, запускать вручную.

mput avatar Sep 24 '19 12:09 mput