mirror of
https://github.com/haproxy/haproxy.git
synced 2026-07-10 17:55:32 -04:00
DEV: patchbot: only display the first 8 chars of the commit id
The commit id column doesn't need to show more than 8 chars to stay unambiguous within a single page, and longer ids needlessly widen the table. Everything that is keyed on the id (the commit link, the row's name= attribute and the cid[] JS array) keeps the full id produced by the pipeline, whatever its length, so this is a display-only change which also gets us ready for a possible future move of the pipeline to longer ids.
This commit is contained in:
parent
b39d2afe48
commit
e4e99282ca
1 changed files with 4 additions and 1 deletions
|
|
@ -404,7 +404,10 @@ for patch in "${PATCHES[@]}"; do
|
|||
|
||||
echo -n "<TD nowrap align=center ${bkp[$cid]:+style='background-color:${BG_B}'}>$seq_num<BR/>"
|
||||
echo -n "<input type='radio' name='review' id='rv_$seq_num' onclick='updt($seq_num,\"r\");' ${do_check:+checked} title='Start review here'/></TD>"
|
||||
echo -n "<TD nowrap ${bkp[$cid]:+style='background-color:${BG_B}'}><tt><a href='${GITURL}${cid}'>$cid</a></tt>${date:+<br/><small style='font-weight:normal'>$date</small>}</TD>"
|
||||
# only the first 8 chars of the commit id are displayed (enough to be
|
||||
# unambiguous on one page); everything keyed (href, name=, cid[])
|
||||
# carries the full id produced by the pipeline, whatever its length.
|
||||
echo -n "<TD nowrap ${bkp[$cid]:+style='background-color:${BG_B}'}><tt><a href='${GITURL}${cid}'>${cid:0:8}</a></tt>${date:+<br/><small style='font-weight:normal'>$date</small>}</TD>"
|
||||
echo -n "<TD nowrap><a href='${GITURL}${cid}'>${pnum:+$pnum }$subj</a>${author:+<br/><div align=right><small style='font-weight:normal'>$author</small></div>}</TD>"
|
||||
echo -n "<TD nowrap align=center>"
|
||||
echo -n "<input type='radio' onclick='updt($seq_num,\"n\");' id='bt_${seq_num}_n' class='n' name='$cid' value='n' title='Drop' $( [ "$verdict" != no ] || echo -n checked) />"
|
||||
|
|
|
|||
Loading…
Reference in a new issue