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

Usability: Python Analyzer should not give Trailing whitespace feedback for web sessions #58

Open
joshgoebel opened this issue May 27, 2022 · 2 comments

Comments

@joshgoebel
Copy link

joshgoebel commented May 27, 2022

Recommended
Line 25 [C0303 trailing-whitespace] : ["Trailing whitespace"] was reported by Pylint.

Which means this code doesn't follow general [code style](https://www.python.org/dev/peps/pep-0008/) conventions. While this type of issue generally doesn't affect the way code executes, it can hurt readability or the performance of automated tools such as documentation generators or test runners.

I find these warnings quite frustrating when using the web editor.

  • There is no auto-correction of trailing whitespace. (it won't fix itself)
  • There is no visual indication of trailing whitespace in the editor. (I'll never know until AFTER)
  • On the web there is no value IMHO in going back and fixing these one by one by line number.
  • It's the kind of problem that keeps breaking over and over (since you can't see it or auto-correct it)

To me this type of feedback is a LOT more useful if someone is using VS Code (or antoher editor)... in that case it makes sense to say "hey, this is problem" because they have so many remedies and it's a more "real" environment - and this is an indication their environment isn't setup for serious Python editing. For students just trying to get started and learning Python via the web editor I think this only gets in the way. For some reason I'm constantly adding trailing whitespace (it happens over and over) so even if I fixed it it would come right back in the next iteration.

@joshgoebel joshgoebel changed the title Usability: Python Analyzer should not give Trailing whitespace feedback for web sesssions Usability: Python Analyzer should not give Trailing whitespace feedback for web sessions May 27, 2022
@MondoBurrito
Copy link

I agree 100%. These errors are so incredibly annoying.

@BethanyG
Copy link
Member

@ErikSchierboom or @iHiD -- could you move this to the Python Analyzer repo? I'll take a look at either disabling the rule, or changing the messaging to being non-essential and suggestive. Many thanks!

@ErikSchierboom ErikSchierboom transferred this issue from exercism/exercism Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants