ThreadА блек также форматировал?

29 messages · 10:52–12:07

🍉

Photo
click to show

А блек также форматировал?

10:52

Ю
Юрий Чебышев
кажется да
10:52
1. 🍉
  🍉
  Там ПР висит от @abc_qwerty, я смержу?
  10:53
  1. Ю
    Юрий Чебышев
    ой, я забыл
    10:53
  2. Ю
    Юрий Чебышев
    @abc_qwerty а разве в прекоммит и в пайплайн не надо было добавить ruff format . и ruff format --check . соответственно?
    10:56
    1. A
      abc <
      Если хочется, чтобы оно форматировало, то пожалуй, до этого там был просто check же
      11:00
      1. 🍉
        🍉
        так ты блек убрал
        11:00
      2. Ю
        Юрий Чебышев
        в пайплайне чек нужен, но сейчас ты блэк выпилил а вместо него чек не сделал
        11:00
      3. Ю
        Юрий Чебышев
        мне кажется нужно заменить black . на ruff format . а black --check . на ruff format --check . или я не понял задумки
        11:02
        A
        abc <
        Можешь показать где был black .?
        11:06
        Ю
        Юрий Чебышев
        Photo
        click to show
        вот в самом начале
        11:07
        🍉
        🍉
        Только в пайплайне не точка, а список файлов передается
        11:09
        Ю
        Юрий Чебышев
        а был ли смысл, если мы отформатировали весь проект?
        11:10
        🍉
        🍉
        Если уже отформатировали, то можно выкинуть
        11:11
        🍉
        🍉
        я это взял со своего легаси тырпрайза, где просто так не применишь форматтер на все файлы 😬
        11:11
        Ю
        Юрий Чебышев
        я думал мы и тут не будем так делать, но чот оно работает не то чтобы супер хорошо, проще отформатировать
        11:12
        A
        abc <
        Стоит ли вообще форматировать в пайплайнах? Не лучше ли просто делать проверки?
        11:19
        🍉
        🍉
        В пайплайне ничего не форматировалось
        11:19
        Ю
        Юрий Чебышев
        нет, в пайплайне мы cделаем ruff format --check
        11:20
        A
        abc <
        Стоит ли в pre-commit использовать ruff check --fix перед ruff format?
        11:38
        🍉
        🍉
        а в чем разница?
        11:38
        A
        abc <
        Первое линтер, а второе форматтер
        11:38
        🍉
        🍉
        Я там оставил комменты по пайплайну. Уже можно убирать Get changed files и заменить ${{ steps.changed-python-files.outputs.all_changed_files }} на . в местах, где используется
        12:05
        A
        abc <
        Не весь, миграции я не трогал 🌚 Ну ок
        12:06
        🍉
        🍉
        Спасибо)
        12:07
        K
        KarmaBot
        🍉, Вы увеличили карму abc < до 2975.11 (+120.97)
        12:07
        Ю
        Юрий Чебышев
        да, думаю в этом порядке норм
        11:39
      4. 🍉
        🍉
        В пайплайне был блек --check --diff --verbose В хуках был блек с активным форматтером
        11:03
S
sanspie
без запятой в сете не будет бить
10:53