Comment #4 on issue 1277 by Comment Types

I think that while this is a good idea in spirit, I'm not sure it would  
work in practice. I don't think severity is
useful in the case of code reviews in the same way that severity isn't all  
that useful in the case of bugs (wait
for the explanation).  In the case of bugs, in most cases, there are a few  
people who have a the big picture of
the product and set the relative severity of the bugs.  In this case  
however, it sounds like the commenter
would set the priority.  That's probably not a good way to go.

I do like the idea of having "comment types" - a defect checkbox, or a  
checkbox that additionally says that
security might be a concern.  This classifies them without priority. I  
think it's up to the developer to read _all_
of the comments and then prioritize and address them.

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:

You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to
To unsubscribe from this group, send email to
For more options, visit this group at

Reply via email to