#30196: Add the tor version to the sbws bandwidth file header
---------------------------+-----------------------------------
 Reporter:  teor           |          Owner:  (none)
     Type:  enhancement    |         Status:  merge_ready
 Priority:  High           |      Milestone:  sbws: 1.2.x-final
Component:  Core Tor/sbws  |        Version:
 Severity:  Major          |     Resolution:
 Keywords:  sbws-roadmap   |  Actual Points:
Parent ID:  #33121         |         Points:  1
 Reviewer:  ahf            |        Sponsor:
---------------------------+-----------------------------------

Comment (by juga):

 Replying to [comment:16 ahf]:
 > I think both the spec change and the sbws changes are good, but I have
 two questions:
 >
 > 1. What is the purpose of the `xxx`'s with 'tech-dept'? Is the goal we
 go back here and do something actionable and is that something that is
 best to have in the code rather than in tickets?

 If i'd create a ticket with a tech-debt (i guess i wrote a typo there)
 changes, i don't know if it'll ever be solved and then in the ticket would
 need to point to all the parts of the code where i detected it while
 working on something else.

 > 2. The constant renaming seems OK to me, but it seems complicated to
 maintain all these lists and how they are subsets/supersets of each other?

 agree, do you have a suggestion on how to change that without having to
 increase minor version because changing API?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30196#comment:17>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
[email protected]
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs

Reply via email to