Comment #4 on issue 1277 by vi...@djgraphite.com: Comment Types http://code.google.com/p/reviewboard/issues/detail?id=1277
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: 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-issues@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 -~----------~----~----~----~------~----~------~--~---
