Автоматизируем форматирование и проверку линтера на пре-коммит

Поділитися
Вставка
  • Опубліковано 16 лис 2024

КОМЕНТАРІ • 24

  • @pika4u380
    @pika4u380 4 місяці тому +3

    Круто. Спасибо за видео. Было бы интересно раскрытие этой темы с примером изменения сообщения коммита. Допустим, добавить префикс к сообщению перед коммитом

  • @kotegav7798
    @kotegav7798 4 місяці тому +1

    Спасибо! как раз на днях преттиера в проект вкручивал, куда только пришёл. Теперь ещё интереснее будет)

  • @vladimirgusar4173
    @vladimirgusar4173 4 місяці тому +10

    Михаил, ты крут! Спасибо за за твой труд!

  • @itsverkunov4517
    @itsverkunov4517 3 місяці тому

    Отличная подача материала! Спасибо!

  • @galibibr02
    @galibibr02 4 місяці тому +1

    Долго ждал такой контент, спасибо большое 👍

  • @Сергей-ю4ж2х
    @Сергей-ю4ж2х 4 місяці тому

    Спасибо за это видео! Очень доходчиво!

  • @ALEKSEY_77737
    @ALEKSEY_77737 4 місяці тому +1

    отличный контент. Большое спасибо!

  • @BOCbMOU
    @BOCbMOU 4 місяці тому +1

    Как альтернатива lint-staged можно использовать банально нпм скрипты. А именно добавить скрипты "husky:pre-commit" и/или "husky:pre-push" и прописать там вызов всех необходимых скриптов с нужными параметрами. И в самом хаски прописать вызов соответствующих скриптов.
    Это нам даёт возможность:
    1. прописать все хуки в одном месте.
    2. без проблем разделить пре-коммит и пре-пуш
    3. использовать более явно те же самые скрипты в пайплайне пулреквестов.
    Но, конечно, это всё дело вкуса в первую очередь.

    • @mishanep
      @mishanep  4 місяці тому +1

      Как тогда выполнить проверку не для всех файлов, а только для тех что в статусе staged?

    • @BOCbMOU
      @BOCbMOU 4 місяці тому

      @@mishanep к примеру, так:
      "prettier --check $(git diff --name-only --diff-filter d | grep -E '\\.(js|cjs|ts|jsx|tsx|json|html)$' | xargs)"
      Да, это куда как длиннее, но это можно вынести в отдельный скрипт и передавать только регексп.
      Но, откровенно говоря, я не понимаю зачем это надо, потому что еслинт, тс и тесты стоит вызывать для всего приложения в любом случае, а те же преттиер и стайллинт работают и без того дай бог секунду.

  • @zatochiSiberian
    @zatochiSiberian 4 місяці тому

    Спасибо, полезно

  • @karatsergio4472
    @karatsergio4472 4 місяці тому

    Полезно , спася 😊

  • @DubinArtur
    @DubinArtur 3 місяці тому +1

    Кто пользовался rush? Какие способности библиотеки вам понравились больше всего?

  • @testwebdevelopment4287
    @testwebdevelopment4287 3 місяці тому

    Миша, привет! Спасибо за видео.
    Ты говоришь про проверку типов TS. Что ты имеешь ввиду?
    `"src/**/*.{js,ts,tsx}":` это?

    • @mishanep
      @mishanep  3 місяці тому

      Можно без js проверку сделать, только ts, tsx

  • @husniddinnorqulov7368
    @husniddinnorqulov7368 3 місяці тому

    👍

  • @alexandershestakov2914
    @alexandershestakov2914 4 місяці тому +2

    А для php что то есть похожее?

    • @mishanep
      @mishanep  4 місяці тому +1

      Про PHP не подскажу

    • @eugeneegorov732
      @eugeneegorov732 2 місяці тому

      Через husky можно cmd запускать. Так на проекте запускаю csfixer на прекоммит

  • @shakapaker
    @shakapaker 4 місяці тому

    я оыбчно для всех вообще файлов измененных претиер запускаю

  • @snatvb
    @snatvb 4 місяці тому

    лучше посмари в сторону biome, эти штуки сильно медленно работают и заставляют реже коммитаться разрабов
    линт стейджед уже 2 года как не обновлялся

    • @CJIu3eHb
      @CJIu3eHb 4 місяці тому +2

      biome вместо преттиера если только. Они сами рекомендуют husky + lint-staged судя по докам.

  • @seryozhamangushev9638
    @seryozhamangushev9638 3 місяці тому +1

    Ты чет запоздал с ним.

    • @mishanep
      @mishanep  3 місяці тому +1

      В каком смысле? Я не подаю это как что-то мега новое, но как полезное. Многие не знают.