search

andreikop / enki

A text editor for programmers
http://enki-editor.org
GNU General Public License v2.0
161 stars 39 forks source link

Flake8 frustrations #349

Closed bjones1 closed 8 years ago

bjones1 commented 8 years ago

When Enki used PyFlakes, it would flag syntax errors as red, and less important things (incorrect spacing, etc.) as warnings. With flake8, its reports as errors things I consider warnings (a line over 79 characters, for example). Is there any way to return to this more helpful color scheme?

andreikop commented 8 years ago

Yes. I also dislike this behavior. I like flake8 output more, but message classification is bad. There are W... codes for warnings and E... codes for errors. I don't know why message for >79 line is an error. It is possible to create a mapping which fixes flake8 error code types.

bjones1 commented 8 years ago

I would suggest treating all F* messages as errors, and all others as warnings (in terms of the color displayed by Enki). That is, F* is red, E*, W*, C*, and N* are all yellow.

andreikop commented 8 years ago

For missing : after def I have a message E901 Syntax Error: invalid syntax

bjones1 commented 8 years ago

Good point. Probably E902 as well, though I'm not sure what error would generate this.

andreikop commented 8 years ago

Need to classify Fxxx errors too.

bjones1 commented 8 years ago

Ugh. I'm avoiding this for now...

andreikop commented 8 years ago

@bjones1, beep if you faced incorrect classification again

bjones1 commented 8 years ago

Thanks! Will do.