[Tracker-discuss] [issue610] Hyperlink git hashes of 10 or 11 characters

2016-12-22 Thread Ezio Melotti
Ezio Melotti added the comment: I'm +0.5 on the prefix idea -- by default it should work without prefix (both for backward compatibility and because most people will copy/paste CSIDs without bothering adding the prefix). It could be still useful in case people want to explicitly disambiguate

[Tracker-discuss] [issue610] Hyperlink git hashes of 10 or 11 characters

2016-12-22 Thread Maciej Szulik
Maciej Szulik added the comment: Still LGTM. ___ PSF Meta Tracker ___

[Tracker-discuss] [issue610] Hyperlink git hashes of 10 or 11 characters

2016-12-22 Thread Brett C. 
Brett C. added the comment:

I went ahead and implemented the idea of hg or git prefixes on the commit 
numbers.

___
PSF Meta Tracker 
<

[Tracker-discuss] [issue589] Add GitHub PR to b.p.o issues using GitHub webhooks (depends on issue586)

2016-12-22 Thread Brett C.
Brett C. added the comment: LGTM ___ PSF Meta Tracker ___

[Tracker-discuss] [issue610] Hyperlink git hashes of 10 or 11 characters

2016-12-22 Thread Brett C.
Brett C. added the comment: Do people think it would be worth adding prefix support? E.g. /lookup/git1234567890 so as to help future-proof number collisions and ambiguity? We could also add "hg" prefix support (svn already has "r"). ___ PSF

[Tracker-discuss] [issue610] Hyperlink git hashes of 10 or 11 characters

2016-12-22 Thread Maciej Szulik
Maciej Szulik added the comment: SGTM ___ PSF Meta Tracker ___