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

Re: [tor-bugs] #2930 [Tor Relay]: Overwrite files in stats/ directory rather than appending to them

#2930: Overwrite files in stats/ directory rather than appending to them
-------------------------+--------------------------------------------------
 Reporter:  karsten      |          Owner:                    
     Type:  enhancement  |         Status:  new               
 Priority:  normal       |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor Relay    |        Version:                    
 Keywords:               |         Parent:                    
   Points:               |   Actualpoints:                    
-------------------------+--------------------------------------------------

Comment(by nickm):

 Hm.  For future patches, the important thing for making it easier to
 review isn't separating tests from other stuff: the important thing is to
 separate code movement and refactoring from behavioral changes.  So I'd
 suggest that maybe you divide future things of this kind into one untested
 patch that does *only* the code movement without adding new stuff, and
 another that changes the behavior and adds tests.  Does that make sense,
 or am I missing something that makes it unreasonably hard?

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