On 12/30/2014 03:35 PM, Joseph Martinot-Lagarde wrote:
Le 30/12/2014 19:48, Adrian Klaver a écrit :
On 12/30/2014 10:14 AM, Gonzalo A. PEÑA CASTELLANOS wrote:
Not sure how pep8 is executed in spyder right now, but it is probably
being called in a QProcess or similar.
If this is the case then we should simply pass this argument to the
QProcess command list
*
*
*--max-line-length=n set maximum allowed line length (default: 79)*
And it would only affect pep8 when being executed on spyder, and no conf
would be created
Should be doable and a nice addition I think
Agreed, but what happens when someone wants to change tabs settings?
Basically this means anytime a pep8 element is exposed in Spyder a
change needs to be made. Seems saner to just point to a single conf file
and let the end user decide how they want to tweak PEP8.
That's true for advanced end-users. Some people would just like that "it
works".
Spyder could try to override this value only if no config file for pep8
is present. Or make the command line configurable, and by default use
the value from spyder's configuration.
And the code paths multiply:) The way I see it, if you are not an
advanced user you use PEP8 as is. If you do not agree you are advanced
user and you tinker as you see fit. Though in the end I do not have to
support the code so it really does not matter what direction this takes.
Cheers
--
Adrian Klaver
[email protected]
--
You received this message because you are subscribed to the Google Groups
"spyder" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/spyderlib.
For more options, visit https://groups.google.com/d/optout.
