Skip to content

fix: задать набор правил линтера явно - #12

Merged
mokevnin merged 1 commit into
mainfrom
fix/pin-ruff-rule-set
Aug 13, 2026
Merged

fix: задать набор правил линтера явно#12
mokevnin merged 1 commit into
mainfrom
fix/pin-ruff-rule-set

Conversation

@mokevnin

Copy link
Copy Markdown
Contributor

Без select ruff берёт набор по умолчанию, а он меняется вместе с версией: в 0.16 добавились сортировка импортов и часть проверок pycodestyle. Из-за этого PR Dependabot с обновлением ruff (#10) роняет CI на коде, который никто не трогал.

Набор задан такой же, как у остальной python-пачки: E, F, C90 в ruff.toml.

Найденное этим набором исправлено: пустые строки и пробелы вокруг = автоправкой ruff, три длинные строки разбиты руками. Склейка user_agent в тесте даёт ровно ту же строку, что была (проверено сравнением).

🤖 Generated with Claude Code

Без select ruff берёт набор по умолчанию, а он меняется вместе с версией: на
0.16 добавилась сортировка импортов и часть проверок pycodestyle, и обновление
ruff роняло CI на коде, который никто не трогал. Набор задан такой же, как у
остальной python-пачки: E, F, C90.

Найденное этим набором исправлено: пустые строки и пробелы вокруг равно
автоправкой ruff, три длинные строки разбиты руками. Склейка user_agent в тесте
даёт ровно ту же строку, что была.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@mokevnin
mokevnin merged commit e28c7bb into main Aug 13, 2026
3 checks passed
@mokevnin
mokevnin deleted the fix/pin-ruff-rule-set branch August 13, 2026 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant