msherry / flycheck-pycheckers

Multiple syntax checker for Python in Emacs, using Flycheck
GNU General Public License v3.0
63 stars 23 forks source link

Default line width does not match checker's defaults #35

Closed rafalcieslak closed 5 years ago

rafalcieslak commented 5 years ago

pycheckers uses the default max-line-width of 80 - which is different to flake8 default (79), PEP8 recommendation (79) and pylint default (also 79).

This discrepancy means that if I use pycheckers on a project that does not customize this value in .pycheckers file, and there are 80 characters in some lines, then

Of course I can circumvent this by setting the line width limit to 79 in the .pycheckers configuration, but I suppose a wrapper around other checkers should not introduce any code style bias by using different defaults than the checkers it wraps.

msherry commented 5 years ago

Thanks for reporting! That's fair -- I've deliberately made pycheckers opinionated in some areas, but it doesn't make sense to be opinionated for a small thing like this. If you want, I'll review and accept a patch modifying this behavior, or I can try to get to it in my spare time.