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

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu May 2 21:54:49 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 atagar):

 >  all, see

 Gotcha. And why do you feel strongly we should change this?

 > don't put stuff in init

 Do you have a use case where this causes any problems? Putting code in
 init is common, and done by lots of projects.

 > flexibility... and yes, it's worth it

 Sorry, but this isn't terribly persuasive. Personally I think our present
 tests provide both great coverage and readable output but if you have a
 practical reason to think pytest would be better I'm all ears.

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


More information about the tor-bugs mailing list