Can't configure column limit about pep8speaks HOT 7 CLOSED

Hi @shonin,

The project reports (and fixes) the errors raised by pycodestyle. So, we can ignore the one error when the column length exceeds 79 chars. If I understood correctly, you want to raise the warning if the limit exceeds 120 columns.

In that case, I think the .pep8speaks.yml can have another custom_errors option. Please feel free to submit a PR.

from pep8speaks.

@daneah @shonin Well I did not know about that. Awesome! I guess a column_limit option can be added. I'll add the feature very soon and inform you about it.

from pep8speaks.

This works great @OrkoHunter, thanks for making the feature!

from pep8speaks.

pycodestyle makes available a --max-line-length option that could likely be pretty easily threaded through from the .pep8speaks.yml file! Would it make more sense to have a line_length or column_limit option in that case?

from pep8speaks.

After looking into so many arguments that are made available by pycodestyle, I thought it'd be good to create a separate anchor for it in the configuration file. Now options like ignore, max-line-length can be added under the new anchor pycodestyle in .pep8speaks.yml file.

Have a look on the updated README. And do let me know if that works for you!

from pep8speaks.

@shonin If your problem is fixed, please close the issue. :)

from pep8speaks.

thanks @OrkoHunter, we really appreciate the quick update! I'll look through this in a more detailed manner on Monday, but it looks like it should work great!

from pep8speaks.