[vtk-developers] gitlab-push woes

[Ken Martin]

Two minor suggestions

1) Make @buildbot and Do: consistent. It seems odd to do "Do: check" or
"Do: merge" and then "@buildbot test".  I would expect "Do: test"

2) Recognition is better than remembering. It is easier for infrequent
developers to **recognize** what they want to do rather than
**remembering** what to do. A good example of this is if you add "Do:
oops" to a merge request it will respond with a list of valid options so
that you can **recognize** what you really wanted, as opposed to just
responding with an error and forcing the user to again try to **remember**
the correct option.  In that vein it might be nice to have the frequently
used commands shown.  Ideally the GUI has them as options and the user
just picks (recognition), but that means messing with the GUI. A not quite
as good alternative would be to maybe have a small part of the GUI on the
merge request page show the basic commands

Do: ***
@buildbot ***
Add reviewer with @...

Or maybe have the "Do: check" output include that help  sort of a

"Hey passed basic content checks, the next steps are often...  Do: ****,
@buildbot **** etc"

I'm sure those of us using this every day will learn the commands and
remember them, but less frequent contributors  might get lost.

Thanks
Ken

Ken Martin PhD
Chairman & CFO
Kitware Inc.
28 Corporate Drive
Clifton Park NY 12065
ken.martin at kitware.com
518 881-4901 (w)
518 371-4573 (f)

This communication, including all attachments, contains confidential and
legally privileged information, and it is intended only for the use of the
addressee.  Access to this email by anyone else is unauthorized. If you
are not the intended recipient, any disclosure, copying, distribution or
any action taken in reliance on it is prohibited and may be unlawful. If
you received this communication in error please notify us immediately and
destroy the original message.  Thank you.