Repository navigation
build(ruff): ignore PLR0917 - #23
Conversation
PLR0913 (max-args = 3) already caps the total argument count, which bounds the positional count, so PLR0917 adds no signal. Refs furedea/agent-config#74 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
@codex review |
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Review summaryNo findings. This PR adds Blocking findings (P0/P1): 0 |
概要
グローバルの
[tool.ruff.lint] ignoreにPLR0917を追加します.理由
PLRプレフィックス経由で既にこのルールを有効にしています.max-args = 3) が引数の総数を既に制限しており,位置引数の数もその範囲に収まります.そのため PLR0917 は追加の情報を与えません.範囲外
検証
ruff check: All checks passedruff format --check: 2 files already formattedRefs furedea/agent-config#74
🤖 Generated with Claude Code