
+1 on including it as a build report, but please at the end of the build process.
In case we agree to add this feature, it's probably best if I include it as part of this PR: https://github.com/scikit-image/scikit-image/pull/538 … this actually shouldn't be more than 2 extra lines… any preferences regarding the style guide checker (pep8.py, pyflakes, …)?
Johannes Schönberger
Am 17.05.2013 um 09:53 schrieb Ankit Agrawal aaaagrawal@gmail.com:
@Josh : I agree that most of PEP8 tests are flexible in order to give priority to readability. Perhaps, I was thinking to have tests for style-guidelines that are rigid, for eg : trailing whitespaces, spaces around '/', '*' etc, file ending with a newline etc.
How about only including it as part of the Travis build report (but not failing)?
@Stefan : This seems a good middle way.
-- You received this message because you are subscribed to the Google Groups "scikit-image" group. To unsubscribe from this group and stop receiving emails from it, send an email to scikit-image+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.

On Fri, May 17, 2013 at 10:05 AM, Johannes Schönberger jschoenberger@demuc.de wrote:
In case we agree to add this feature, it's probably best if I include it as part of this PR: https://github.com/scikit-image/scikit-image/pull/538 … this actually shouldn't be more than 2 extra lines… any preferences regarding the style guide checker (pep8.py, pyflakes, …)?
That'd be great. Both of those, please!
pyflakes $1 echo "## pyflakes above, pep8 below ##" pep8 --repeat $1
Stéfan
participants (2)
-
Johannes Schönberger
-
Stéfan van der Walt