This passed off the port bow this evening: https://github.com/hayd/pep8radius. I like the approach, applying pep8 only in the vicinity of where one has recently been working.
On Sun, Nov 19, 2017 at 6:31 PM, Edward K. Ream <edream...@gmail.com> wrote: > > > On Sun, Nov 19, 2017 at 5:09 PM, Terry Brown <terrynbr...@gmail.com> > wrote: > >> >> >> > I have just created #584: clean up the worst pep8 offenses >> > <https://github.com/leo-editor/leo-editor/issues/584>. >> >> >> >> I occasionally fix PEP 8 things when I see them, missing spaces after >> commas particularly bug me, but I like to keep those changes at least >> within a file I'm changing anyway, if not within the part of the file >> I'm changing. > > > The beautify* commands will fix such things. > > >> I feel that PEP-8 is seen as a place to start, not a set >> of commandments that must be obeyed. 80 char line length is >> particularly dated in today's monitor-scape. >> http://jakevdp.github.io/blog/2017/11/09/exploring-line-leng >> ths-in-python-packages/ > > > I agree completely. > > Hmm, I guess I don't see any harm in hunting PEP-8 things, but it >> wouldn't be a priority for me. >> > > I've just closed #584. It turns out that the KeyHandlerClass class uses > mixedCase method names consistently. True, there are other classes in > leoKeys.py that use the other way, but that kind of inconsistency can't be > helped now. > > Furthermore, I think that using the "reloadSettings" name in all classes > makes sense, because it creates a different kind of consistency. > Alternatively, we could replace all the reloadSettings methods using a > publish/subscribe pattern, but I see no urgent reason not the use the > reloadSettings pattern. > > In short, there are far bigger issues facing Leo than naming conventions > ;-) > > Edward > > -- > You received this message because you are subscribed to the Google Groups > "leo-editor" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to leo-editor+unsubscr...@googlegroups.com. > To post to this group, send email to leo-editor@googlegroups.com. > Visit this group at https://groups.google.com/group/leo-editor. > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "leo-editor" group. To unsubscribe from this group and stop receiving emails from it, send an email to leo-editor+unsubscr...@googlegroups.com. To post to this group, send email to leo-editor@googlegroups.com. Visit this group at https://groups.google.com/group/leo-editor. For more options, visit https://groups.google.com/d/optout.
