Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add code action for E242 #176

Merged
merged 1 commit into from
Jun 30, 2023
Merged

Conversation

kyoto7250
Copy link
Contributor

@kyoto7250 kyoto7250 commented Jun 28, 2023

close #29

This PR adds the rule for detecting tab after ',' (E242)

@kyoto7250
Copy link
Contributor Author

@microsoft-github-policy-service agree

@karthiknadig karthiknadig changed the title Enable detect E242 Add code action for E242 Jun 28, 2023
@kyoto7250
Copy link
Contributor Author

It looks like we need a label to represent this change, is there anything I can do?

@karthiknadig karthiknadig self-assigned this Jun 29, 2023
@karthiknadig karthiknadig added the feature-request Request for new features or functionality label Jun 29, 2023
karthiknadig
karthiknadig previously approved these changes Jun 29, 2023
eleanorjboyd
eleanorjboyd previously approved these changes Jun 29, 2023
@karthiknadig
Copy link
Member

@kyoto7250 Thanks for the PR. Everything is looking good. Can you fix the merge conflict and then we can merge this?

),
(
"E271",
"from collections import (namedtuple, defaultdict)",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I rearranged in order

@kyoto7250
Copy link
Contributor Author

@karthiknadig

I rebased the commit.
Could you please check again?

@karthiknadig karthiknadig enabled auto-merge (squash) June 30, 2023 17:09
@karthiknadig karthiknadig merged commit 7637c71 into microsoft:main Jun 30, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature-request Request for new features or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add quick fix for "tab after ‘,’" error
3 participants