[tor-bugs] #7631 [Stem]: Try a PEP8 style checker

[Tor Bug Tracker & Wiki]

#7631: Try a PEP8 style checker
----------------------------+-----------------------------------------------
    Reporter:  atagar       |       Owner:  atagar  
        Type:  enhancement  |      Status:  reopened
    Priority:  minor        |   Milestone:          
   Component:  Stem         |     Version:          
  Resolution:               |    Keywords:  testing 
      Parent:               |      Points:          
Actualpoints:               |  
----------------------------+-----------------------------------------------

Comment(by neena):

 Replying to [comment:7 atagar]:
 > Reopening. This is understandably a pain point for new contributors so
 we really should change it. I'd be fine with replacing our style checker
 [1] with something that gets us closer to PEP8 compliance. After all, it's
 easier for me to change my habits and vimrc than ask it of the world. ;)
 >
 > My one caveat to PEP8 is the four space indentations. From what I've
 seen python projects very widely on this.
 By complying with pep8, we would be helping maintain consistency among
 them. :p

 > If active contributors really want us to move to four spaces then fine,
 but I'd like to have some discussion around that first.

 I don't mind the current style. But, I think it would be better for Stem
 to comply with PEP8 completely simply because it's The Convention To
 Confirm To.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/7631#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online