-
Notifications
You must be signed in to change notification settings - Fork 507
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
SA1027 (TabsMustNotBeUsed) code fix ignores trivia #1246
Comments
same issue with tabs inside comments |
Working on it! Question: which points from the contributing guide are important for bug fixes in existing analyzers and code fixes? How should my branch be named? |
@robinsedlaczek I wrote the contributing guide back when only I was working on the project. It could be entirely out of date now that the project took off. The comment you posted is enough to count as claiming it. Then there is a code review after you submit a pull request. 😄 |
@sharwell Cool, thanks! |
@robinsedlaczek I ended up fixing this as part of another efficiency change I was working on. |
@sharwell Oh, so I am too late. :) Great, that you fixed it. I'll have a look to learn. Thanks! |
e.g. assume all lines indented by tabs
after code fix we will receive:
code indented by spaces but documentation still indented by tabs
The text was updated successfully, but these errors were encountered: