-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Pack of fuzzer Autofix/Infinite loop errors [2] #8402
Comments
Ruff 0.1.3 (latest changes from main branch)
file content:
error
|
Ruff 0.1.3 (latest changes from main branch)
file content:
error
|
Ruff 0.1.3 (latest changes from main branch)
file content:
error
|
Ruff 0.1.3 (latest changes from main branch)
file content:
error
|
Ruff 0.1.3 (latest changes from main branch)
file content:
error
|
Ruff 0.1.3 (latest changes from main branch)
file content:
error
|
## Summary Addresses #8402 (comment) ## Test Plan Added associated test
This PR avoids creating the fix for `D301` if the string is prefixed with `u` i.e., it's a unicode string. The reason being that `u` and `r` cannot be used together as it's a syntax error. Refer: #8402 (comment)
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
We ended up with a syntax error here via `from trio import lowlevel.checkpoint`. The new solution avoids that error, but does miss cases like: ```py from trio.lowlevel import Timer ``` Where it could insert `from trio.lowlevel import Timer, checkpoint`. Instead, it'll add `from trio import lowlevel`. See: #8402 (comment)
Ruff 0.1.5 (latest changes from main branch)
file content:
error
|
@AlexWaygood you might be interested in the above |
file content:
error
|
file content:
error
|
file content:
error
|
file content:
error
|
file content:
error
|
@diceroll123 - Are you interested in taking a look at some of the |
## Summary A fix for #8402 (comment) Improves the code, as well. :) ## Test Plan `cargo test`
## Summary Fix for #8402 (comment) ## Test Plan `cargo test`
#9698) Fixes the issues highlighted in #8402 (comment) and #8402 (comment)
Ruff 0.2.2
file content:
error
|
ruff 0.2.2
file content:
error
|
ruff 0.3.5
file content:
error
|
this would be interesting to fix, because as-is, it's an AttributeError at runtime. Syntactically, it's fine though. |
It seems #8402 (comment) |
@diceroll123 Yeah, that's correct. I think it's #9821 which fixed it. Thank you! |
Previous part(#7455) exceeded 50 reported issues, and is hard to search/add new problems, so decided to create new list
Not all errors are reported here, but more can errors with nice format can be obtained:
reports-
- inside, should be grouped available file that causes problems with info which rule is needed to trigger them06.04.2024 results - 11673 broken files -
tmp_folder.7z.zip
Rules that causes problems(for some files, more than one rule needs to be enabled)
The text was updated successfully, but these errors were encountered: