[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

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

#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

_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs