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

Added quick fix for "tab before keyword" warning. #44

Merged
merged 3 commits into from
Sep 21, 2022

Conversation

radhikaaggarwal19
Copy link
Contributor

Added quick fix for "tab before keyword" warning

Added quick fix for "tab before keyword" warning
@ghost
Copy link

ghost commented Sep 16, 2022

CLA assistant check
All CLA requirements met.

Copy link
Contributor Author

@radhikaaggarwal19 radhikaaggarwal19 left a comment

Choose a reason for hiding this comment

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

Edited for missing 'e'

@karthiknadig karthiknadig added feature-request Request for new features or functionality and removed debt labels Sep 19, 2022
@karthiknadig karthiknadig changed the base branch from main to OSD-2022 September 19, 2022 03:37
@karthiknadig
Copy link
Member

nox > black --check ./src/test/python_tests
would reformat src/test/python_tests/test_code_actions.py

@radhikaaggarwal19 looks like there is some formatting issue with test_code_actions.py. Can you install black in your environment (python -m pip install black), and then try running this python -m black src/test/python_tests/test_code_actions.py. After that commit those changes and push.

@radhikaaggarwal19
Copy link
Contributor Author

I made the update and committed the changes. Hope it works now!

@karthiknadig karthiknadig changed the title https://github.com/microsoft/vscode-flake8/issues/41 Added quick fix for "tab before keyword" warning. Sep 21, 2022
@karthiknadig karthiknadig merged commit 6cc1ba4 into microsoft:OSD-2022 Sep 21, 2022
# 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.

2 participants