[tor-bugs] #30359 [Core Tor/Stem]: Stem PEP8 compliant

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu May 2 21:04:15 UTC 2019


#30359: Stem PEP8 compliant
---------------------------+------------------------
 Reporter:  0xrichard      |          Owner:  atagar
     Type:  enhancement    |         Status:  new
 Priority:  Low            |      Milestone:
Component:  Core Tor/Stem  |        Version:
 Severity:  Minor          |     Resolution:
 Keywords:  dev            |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+------------------------

Comment (by 0xrichard):

 > ...  before investing your time.

 I had a great day, found some things I would've done differently, like
 putting stuff in __init__.py

 > Decide which of the above PEP8 compliance rules you feel strongly that
 we should follow.

 [flake8]
 ignore = E501,W504,F811,F821,W605,E402,F401,F405,E131,W503,E265,E999,F403

 > File a ticket to discuss why you think we should change it.
 Just did

 > Once we've established a consensus on following the rule remove its
 ignore configuration from the file mentioned above. Stem's tests should
 now cite all the spots where we don't comply with it.

 That's up to you, you tell me. I got time :)

 > Make the adjustments (like your patch does) to correct the compliance
 issues.

 I'd be happy to

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


More information about the tor-bugs mailing list