My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 2158: Enable verification of multiple conditions (Verified box with multiple choice)
1 person starred this issue and may be notified of changes. Back to list
Status:  New
Owner:  ----


Sign in to add a comment
 
Reported by l.stelm...@samsung.com, Oct 2, 2013
Affected Version:
2.6.1.20130902

What steps will reproduce the problem?
1. Click the Review button.
2. Look for the "Verified" box
3. There are only three options

+1 Verified
 0 No Score
-1 Fails

What is the expected output? What do you see instead?

It should be possible define multiple conditions under which the patch can be tested. For example different hardware architectures. Then for each condition (architecture) there should be a separate radio button group. Then it should be possible to set up custom rules for that say when the patch can be merged (for some projects some radio groups may be only informational, not required)

For example, the table below says that I have not tested the change intel processors but only for ARM and it works only on ARMv7.

|             | i586 | x86_64 | ARMv5 | ARMv7 |
| +1 Verified | ( )  | ( )    | ( )   | (*)   |
| 0 No score  | (*)  | (*)    | ( )   | ( )   |
| -1 Fails    | ( )  | ( )    | (*)   | ( )   |




Oct 2, 2013
Project Member #1 edwin.ke...@gmail.com
You can simply define own review labels for your project:
  http://gerrit-documentation.googlecode.com/svn/Documentation/2.6/config-labels.html
Sign in to add a comment

Powered by Google Project Hosting