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.

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

Add code action for E241 #168

Merged
merged 4 commits into from
Jun 29, 2023
Merged

Add code action for E241 #168

merged 4 commits into from
Jun 29, 2023

Conversation

SruthiSathya
Copy link
Contributor

fix #28 for warning code E241

@karthiknadig
Copy link
Member

@SruthiSathya Thanks for the PR. Can you reply with the CLS comment as mentioned here: #168 (comment)

bundled/tool/lsp_server.py Outdated Show resolved Hide resolved
@karthiknadig karthiknadig added the feature-request Request for new features or functionality label Jun 27, 2023
eleanorjboyd
eleanorjboyd previously approved these changes Jun 28, 2023
@karthiknadig karthiknadig changed the title fix #28 for warning code E241 Add code action for E241 Jun 28, 2023
@karthiknadig
Copy link
Member

@SruthiSathya It looks like you need to run black formatter on this file test_code_actions.py.

@karthiknadig karthiknadig enabled auto-merge (squash) June 29, 2023 17:24
@karthiknadig karthiknadig merged commit f54d9cc into microsoft:main Jun 29, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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 "multiple spaces after ‘,’" error
3 participants