Add rye format and rye lint to the pre-commit configuration#170
Conversation
> Set up a project. > Set up a descent linting/formatting tools. > Configure the CI. > Add modern pre-commit git hooks managers. > Not use them for running formatters and linters. This patch adds the hooks for checking formatting and linting of the python code to the pre-commit configuration.
|
This |
|
О, крутая штука! Я не добавил ruff в pre-commit, потому что идея содержать отдельные бинарики ruff и в rye, и в pre-commit показалась мне избыточной. Я не знал, что можно просто команды дёргать. Думаю, для тестирования стоит порекламить этот патч в чатике. |
|
Надо бы в CI убрать запуск проверки пре-коммит хуков или форматтера, так как не имеет смысла дважды запускать проверку форматирования (и в хуках и отдельной командой) formal-lang-course/.github/workflows/code_style.yml Lines 31 to 39 in 60ff186 |
|
@georgiy-belyanin @WoWaster Что в итоге надо стдеать, чтобы таки влить этот реквест? |
|
Да, надо сделать как предлагает Вадим --- убрать запуск rye в CI скриптах и запускать всё через pre-commit. |
This patch adds the hooks for checking formatting and linting of the python code to the pre-commit configuration.