Differential D5397

pep8: Make pep8 warnings less annoying
ClosedPublic
Actions

Authored by kfunk on Apr 11 2017, 9:13 AM.

Details

Reviewers
brauch
flherne
Group Reviewers
KDevelop
Commits
R53:e4aedbb2223d: pep8: Make pep8 warnings less annoying
Summary

Don't hard-code the length of the warnings range to 8, it clutters up
the text editor a lot. Just use a one-char warning indicator.

That also makes the warning show up at the "correct location", not
somewhere seemingly random.

Diff Detail

Repository
R53 KDevelop: Python Support
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kfunk created this revision.Apr 11 2017, 9:13 AM
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptApr 11 2017, 9:13 AM
flherne edited edge metadata.Apr 11 2017, 9:33 AM

(as on IRC) I like this - the error location is clearer than with the large underline, and it reduces the clutter when editing oddly-formatted files.

brauch edited edge metadata.Apr 12 2017, 1:36 PM

I don't have a strong opinion on this, if you have, feel free to change it.

Closed by commit R53:e4aedbb2223d: pep8: Make pep8 warnings less annoying (authored by kfunk). · Explain WhyApr 12 2017, 1:36 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mpythonstylechecking.cpp (4 lines)

Diff 13355

View Options

pythonstylechecking.cpp

Loading...