[tor-bugs] #33688 [Core Tor/Tor]: README cleanups
    Tor Bug Tracker & Wiki 
    blackhole at torproject.org
       
    Wed Mar 25 23:47:37 UTC 2020
    
    
  
#33688: README cleanups
--------------------------+------------------------------------
 Reporter:  bduszel       |          Owner:  (none)
     Type:  enhancement   |         Status:  needs_revision
 Priority:  Very Low      |      Milestone:  Tor: 0.4.4.x-final
Component:  Core Tor/Tor  |        Version:
 Severity:  Minor         |     Resolution:
 Keywords:  doc           |  Actual Points:
Parent ID:                |         Points:
 Reviewer:  catalyst      |        Sponsor:
--------------------------+------------------------------------
Changes (by catalyst):
 * status:  needs_review => needs_revision
Comment:
 Thanks for the patch! It mostly looks good. I have a few minor comments:
 We don't consistently use the `.../doc/HACKING` reference style to refer
 to locations in the source tree. I'm not sure if we want to adopt it. Is
 that a syntax that some other markdown language treats specially? I think
 the best alternative might be to make all pathname references be relative
 to the top of the source tree, e.g., `doc/HACKING` instead of
 `.../doc/HACKING`.
 The "underline" of the section headings should be the same length as the
 title. After your patch, some of them are longer than their corresponding
 title strings. (Some of them were already longer, though.) Alternatively,
 we could move toward using the `## Section name` style, which doesn't
 require variable-length underlines.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/33688#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
    
    
More information about the tor-bugs
mailing list