Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ctypes.WinError() returns false positive for RSE102 #6730

Closed
charliermarsh opened this issue Aug 21, 2023 · 4 comments · Fixed by #6731 or #9027
Closed

ctypes.WinError() returns false positive for RSE102 #6730

charliermarsh opened this issue Aug 21, 2023 · 4 comments · Fixed by #6731 or #9027
Assignees
@charliermarsh
Labels
bug Something isn't working

Comments

@charliermarsh
Copy link
Member

Raising ctypes.WinError() also triggers this. There is a code snippet here for example.

Originally posted by @tdulcet in #5416 (comment)
@charliermarsh charliermarsh mentioned this issue Aug 21, 2023
Merged
@charliermarsh charliermarsh added the bug Something isn't working label Aug 21, 2023
@charliermarsh charliermarsh self-assigned this Aug 21, 2023
charliermarsh added a commit that referenced this issue Aug 21, 2023
@charliermarsh
Allow ctypes.WinError() in flake8-raise (#6731)
d5a51b4 
Closes #6730.
@tdulcet
Copy link

tdulcet commented Nov 9, 2023

This issue is occurring again. It seems to have been regressed by #8231.

@tdulcet
Copy link

tdulcet commented Dec 6, 2023

@charliermarsh - Friendly ping regarding the above comment, as the regression is still present in v0.1.7.

@charliermarsh
Copy link
Member Author

Oh thanks @tdulcet. Will take a look.

@charliermarsh charliermarsh mentioned this issue Dec 6, 2023
Merged
@charliermarsh
Copy link
Member Author

Fixed!

charliermarsh added a commit that referenced this issue Dec 6, 2023
@charliermarsh
Avoid removing parentheses on ctypes.WinError (#9027)
b918647 
Re-resolves #6730.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@charliermarsh charliermarsh

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@charliermarsh @tdulcet