New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable ruff's Perflint rule. #2969
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2969 +/- ##
==========================================
- Coverage 99.63% 99.63% -0.01%
==========================================
Files 117 117
Lines 17593 17578 -15
Branches 3173 3172 -1
==========================================
- Hits 17528 17513 -15
Misses 46 46
Partials 19 19
|
src/trio/_subprocess.py
Outdated
@@ -723,7 +723,7 @@ async def read_output( | |||
) -> None: | |||
async with stream: | |||
async for chunk in stream: | |||
chunks.append(chunk) | |||
chunks.append(chunk) # noqa: PERF402 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like PERF402
on an argument passed in is incorrect and probably is a bug...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created astral-sh/ruff#11316 for this
Co-authored-by: EXPLOSION <git@helvetica.moe>
Co-authored-by: Spencer Brown <spencerb21@live.com>
Co-authored-by: Spencer Brown <spencerb21@live.com>
Updating branch to see if any changes with new ruff versions |
pre-commit.ci autofix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No thoughts, I accidentally redid my last comment. I guess the noqa bothers me that much lol
Given that astral-sh/ruff#11316 is still open, I am thinking of merging this soon unless anyone has any other comments. |
This pull request enables ruff's perflint rule and handles the associated fixes or ignores.