[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 0xrichard):

 > Which of these do you strongly feel we should change and why?

 all, see:

 inore = E501,W504,F811,F821,W605,E402,F401,F405,E131,W503,E265,E999,F403

 > If you make a separate commit with this and repro steps for triggering
 the circular dependency bug I'd be happy to chat.

 don't put stuff in __init__

 > What benefit will that provide over what we presently have?

 flexibility

 I recently rewrote +/- 400 django unit tests to pytest, and yes, it's
 worth it

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30359#comment:7>
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