Repository navigation
[CI] Fix ruff docstring violations in EC connector files - #57283
Conversation
Fix D200 and docstring formatting violations in vllm/distributed/ec_transfer/ec_connector/base.py and vllm/distributed/ec_transfer/ec_connector/metrics.py that broke pre-commit on main after #54960 merged without rebasing onto the new pydocstyle rules from #52136. Co-authored-by: Kimi Code <noreply@moonshot.ai> Signed-off-by: khluu <khluu@users.noreply.github.com> Signed-off-by: khluu <khluu000@gmail.com>
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.
Tip: disable this comment in your organization's Code Review settings.
|
/ci run |
github-actions
Bot
added
the
ready
|
❌ This PR is 1 commit behind upstream |
|
/ci run --allow-stale |
|
✅ Triggered Buildkite CI #89531 for commit
|
|
/ci cancel |
|
✅ Requested cancellation of 1 CI build for |
2 participants
What
Fix pre-commit ruff failures on main caused by merge skew between:
pydocstyleto theruffrules #52136 which added pydocstyle (D) rules to ruffThe PR (#54960) passed CI before the pydocstyle rules were enabled on main, but was merged without rebasing, causing D200 violations in:
vllm/distributed/ec_transfer/ec_connector/base.pyvllm/distributed/ec_transfer/ec_connector/metrics.pyChanges
Pure ruff auto-fixes: move opening
"""to the same line as the first text (D200) and collapse single-line docstrings to one line (ruff-format).Testing
pre-commit run ruff-check --all-filespassespre-commit run ruff-format --all-filespassesAI Disclosure
This fix was generated with AI assistance (Kimi Code CLI). The changes are purely mechanical ruff auto-fixes with no semantic modifications.