Comment #6 on issue 1277 by lonico: Comment Types
http://code.google.com/p/reviewboard/issues/detail?id=1277

Just to add my 2 cents. Management needs to be convinced that reviews are 'useful'.

So we need some metrics that could show that reviews are finding problems. I'd be happy with a 3 level system: Major, Minor, Info. And by counting the number of Major and Minor, I could argue that reviews are finding problems that needed to be fixed. I don't see the need for a type, but I understand opinions may vary. My point is
more to be able to make a difference between 'require a fix' and 'for info'.

My concern is about adding some red tape, and breaking the flow. We could set a default severity, but then a lot of people would just keep the default. We could
force a choice, but then, it hinders the flow.

I also would like the severity to be customizable. Though my preference would be 3 choices, we're using a defect management system that uses severities 1 to 5. So even if it's an overkill, it may be easier to use the same scale, as people are used to
it.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.

Reply via email to