you could use pylint to check for style consistency automatically On Feb 1, 9:33 am, Richard <[email protected]> wrote: > > There are few catch-all except left and they should be removed. > > $ grep "except:" gluon/*py | wc -l > 133 > > On Feb 1, 2:28 pm, mdipierro <[email protected]> wrote: > > > > > We have done throught many pep8 revisions but things like this happen. > > I will take patches to clean-up. > > > There are few catch-all except left and they should be removed. > > > On Jan 31, 9:03 pm, Richard <[email protected]> wrote: > > > > Browsing through gluon I noticed many inconsistencies such as spacing > > > around operators (a=1; a =1; a = 1), spacing between arguments (a,b, > > > c ,d), string quotes (a='hello; a="hello"), etc. Is there a > > > recommended style for the source code (PEP8?), and even more > > > importantly for examples? > > > > Also there are a lot of catch all exceptions that could potentially > > > mask other problems, eg: > > > try: > > > from google.appengine.api.urlfetch import fetch > > > return fetch(url).content > > > except: > > > import urllib > > > return urllib.urlopen(url).read() > > > > Richard
-- You received this message because you are subscribed to the Google Groups "web2py-users" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/web2py?hl=en.
