cubicweb-vcreview #3836672 All patches of a single ticket should not be reviewed by the same person [open]

Rational: we promote a 2 step process for code integration:

  • a formal review that every good programer could do without any business context
  • an integration review by an "expert" of the business matter which will ensure the general direction is respected

Chosing the same reviewer for the (sometimes big) set of patches of one ticket goes against this. The integrator should probably read all those patches at once, not the reviewer IMHO.

done in<not specified>
closed by<not specified>