<div dir="ltr">+1</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Mar 21, 2015 at 11:03 AM, Ken Martin <span dir="ltr"><<a href="mailto:ken.martin@kitware.com" target="_blank">ken.martin@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Two minor suggestions<br>
<br>
1) Make @buildbot and Do: consistent. It seems odd to do "Do: check" or<br>
"Do: merge" and then "@buildbot test".  I would expect "Do: test"<br>
<br>
2) Recognition is better than remembering. It is easier for infrequent<br>
developers to **recognize** what they want to do rather than<br>
**remembering** what to do. A good example of this is if you add "Do:<br>
oops" to a merge request it will respond with a list of valid options so<br>
that you can **recognize** what you really wanted, as opposed to just<br>
responding with an error and forcing the user to again try to **remember**<br>
the correct option.  In that vein it might be nice to have the frequently<br>
used commands shown.  Ideally the GUI has them as options and the user<br>
just picks (recognition), but that means messing with the GUI. A not quite<br>
as good alternative would be to maybe have a small part of the GUI on the<br>
merge request page show the basic commands<br>
<br>
Do: ***<br>
@buildbot ***<br>
Add reviewer with @...<br>
<br>
Or maybe have the "Do: check" output include that help  sort of a<br>
<br>
"Hey passed basic content checks, the next steps are often...  Do: ****,<br>
@buildbot **** etc"<br>
<br>
I'm sure those of us using this every day will learn the commands and<br>
remember them, but less frequent contributors  might get lost.<br>
<br>
Thanks<br>
Ken<br>
<br>
Ken Martin PhD<br>
Chairman & CFO<br>
Kitware Inc.<br>
28 Corporate Drive<br>
Clifton Park NY 12065<br>
<a href="mailto:ken.martin@kitware.com">ken.martin@kitware.com</a><br>
<a href="tel:518%20881-4901" value="+15188814901">518 881-4901</a> (w)<br>
<a href="tel:518%20371-4573" value="+15183714573">518 371-4573</a> (f)<br>
<br>
This communication, including all attachments, contains confidential and<br>
legally privileged information, and it is intended only for the use of the<br>
addressee.  Access to this email by anyone else is unauthorized. If you<br>
are not the intended recipient, any disclosure, copying, distribution or<br>
any action taken in reliance on it is prohibited and may be unlawful. If<br>
you received this communication in error please notify us immediately and<br>
destroy the original message.  Thank you.<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">William J. Schroeder, PhD<br>Kitware, Inc.<br>28 Corporate Drive<br>Clifton Park, NY 12065<br><a href="mailto:will.schroeder@kitware.com">will.schroeder@kitware.com</a><br><a href="http://www.kitware.com">http://www.kitware.com</a><br>(518) 881-4902</div>
</div>