Hi all, Currently, if there is a change to a patch on gerrit, all of the previous tags are cleared. For instance, if you upload a patch and receive a +2 review, then you make a minor update (or any update) the +2 is lost.
I'm proposing changing this rule to only clear Maintainer +1 and keep all +2's. Thus, for small changes you will no longer need to re-review the patch. The only downside to this is if someone makes a major change after you have given a +2, you will have to notice and remove your +2 if you disagree. Otherwise, I am hoping this will change will make things a little easier on reviewers. Anyone have any objections (or other opinions)? If I don't hear anything for a week we'll go ahead and make the change. Cheers, Jason _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
