My suggestion is that it is enough to ensure in the culture / workflow. Lets indicate in the workflow doc, developer should ensure there are no PEP8 warnings locally before pushing. And give concrete indications how to do it, for example suggesting to use an IDE that supports PEP8 warnings highlighting (Spider, Komodo, EcipsePyDev) or giving command line instruction on how to run a PEP8 checker.
Pablo On Tuesday, July 30, 2013 3:55:20 PM UTC+2, F. B. wrote: > > The test of code quality already performs a check of no trailing-space and > just 1 new line at the end of the file in sympy's code. > > What about adding more tests? > > The lack of conformance may cause merge problems in old PR. > > What about adding the space requirement in comma separated lists? > > What about a constraint on the number of newlines inside a file? > -- You received this message because you are subscribed to the Google Groups "sympy" 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/sympy. For more options, visit https://groups.google.com/groups/opt_out.
