Skip to content

Conversation

@reswqa
Copy link
Contributor

@reswqa reswqa commented Oct 23, 2023

Summary

Introduce auto fix for E223, E224, E242. This partially address #8120.

Test Plan

Already covered.

@reswqa reswqa changed the title add auto-fix for E223,224 add auto-fix for E223,224,242 Oct 23, 2023
@reswqa
Copy link
Contributor Author

reswqa commented Oct 23, 2023

I'm not clear why CI(benchmarks) failed.

@reswqa reswqa marked this pull request as ready for review October 23, 2023 14:15
@charliermarsh charliermarsh self-requested a review October 23, 2023 16:05
@charliermarsh charliermarsh added the fixes Related to suggested fixes for violations label Oct 23, 2023
Copy link
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@charliermarsh charliermarsh enabled auto-merge (squash) October 23, 2023 16:10
@charliermarsh charliermarsh merged commit 92baa35 into astral-sh:main Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fixes Related to suggested fixes for violations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants