Github user jbfavre commented on the issue:

    https://github.com/apache/trafficserver/pull/1303
  
    Well, at least I won't go against it :)
    
    When fixing it in first place, docutils devs feedback was that we wrongly 
used Inliner, in the sense it's supposed to be internal.
    What if another docutils internal change breaks it again ? Don't think it 
won't happen because the discussion I had showed that not every docutils dev is 
happy with last change. So, chances are that there'll be another refacto, maybe 
for 0.15 or 0.16.
    Better solution, in both docutils devs & myself, should be to make Sphinx 
allow customization.
    
    Second concern is that, IMHO, we shouldn't allow 2 different syntax to 
reference trafficserver issues in documentation.
    This usually leads to confusion (why should I use one more than another ?)
    
    Finally, the fix I made has another positive effect: we now have a 
consistent way of referencing issues, for both Jira & Github. For now, I don't 
see any way to differentiate Jira & github ones with the Autolink feature.
    
    My 2 cents,


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to