Size: a a a

2021 April 18

R🔘

Roman 🔘 in Frontend UA
Я вот раньше болел за такие штуки, но сейчас пре комит хук для меня бе 🤮
источник

R🔘

Roman 🔘 in Frontend UA
Валидации линтером на CI оказалось достаточно, чтобы не пропускать не валидный стиль кода в основную ветку
источник

R🔘

Roman 🔘 in Frontend UA
Пре комит хук НЕ НУЖЕН (мое субъективное мнение)
источник

D

Dmitry in Frontend UA
А настройка on save в IDE ?
источник

VS

V7v S6k in Frontend UA
ну якщо всі користуються одною іде хіба...
источник

EO

Eugene Obrezkov in Frontend UA
проблема такого подхода, что ты узнаешь про заваленный форматинг уже на CI, а не локально - увеличенное время отзыва
источник

EO

Eugene Obrezkov in Frontend UA
это не проблема, если билды быстрые (в пределах минут, за чаём сходить можно)
но становится проблемой на больших репозиториях, которые собираются от получаса и дальше
источник

EO

Eugene Obrezkov in Frontend UA
у меня настроено codeActionOnSave: ['source.fixAll']
а для тех, кого такого нету - прекомит\препуш хук
источник

Вт

Ві тя in Frontend UA
+1 сьекономит время билда
А еще лучше когда и сі проверяет и прекоммит
источник

DZ

Dmitry Zherebko in Frontend UA
меня немного бесит когда пытаешься закоммитить что-то и иде начинает подвисать
источник

R🔘

Roman 🔘 in Frontend UA
Как вариант, или просто смотреть во вкладку problems в IDE
источник

DB

Dima Bildin in Frontend UA
Ну типа да, кому-то ок перед коммитом подвисание, кому-то на сейв, кто-то итак нажимает комбинацию после каждых трёх строк запустить формат текущего файла и ок с тем, что пусть только на ci проверяется, насильно можно не ставить, каждый себе настроит, как им удобно
источник

EO

Eugene Obrezkov in Frontend UA
я на эти все свистелки смотрю просто и не усложняю

У нас в репе, где я за инфру смотрю ничего такого нету, более того, форматинг на CI запускается чисто ради галочки. Идея простая - если проблема с форматингом, то и фиг с ним, не сейчас, так завтра другим коммитом починится. А всё остальное валит билд, конечно, если что-то что может сломаться
источник

EO

Eugene Obrezkov in Frontend UA
но при этом, когда контрибучу в другие проекты и там все эти свистелки настроены... ну ок, настроили то настроили 🤷
источник

Вт

Ві тя in Frontend UA
Приттиер подвисать?
источник

DZ

Dmitry Zherebko in Frontend UA
+ eslint
если сравнить с обычным гитом без пре коммитов то ощущается медленно
источник

DZ

Dmitry Zherebko in Frontend UA
может стоит убрать еслинт с пре коммита
источник

Вт

Ві тя in Frontend UA
Еслинт лучше только как подсветку в иде и в сі использовать, он медленный да)
источник

EO

Eugene Obrezkov in Frontend UA
на препуш лучше перенести
источник

EO

Eugene Obrezkov in Frontend UA
никому нет дела до коммитов, которые у тебя локально происходят
источник