Re: Eliminate "Bouncing" in Reviews

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thursday, 08 February 2007 at 05:20, Warren Togami wrote:
> New thought.
> 
> Possibly this compromise process satisfies both camps, by eliminating 
> "bouncing" back and forth, while being assigned to the next actor, and 
> making logical use of NEEDINFO.
> 
> 1) State 1: Not Yet Reviewed
> ASSIGNED pointer to nobody@xxxxxxxxxxxxxxxxx
> 
> 2) State 2: Under Review
> ASSIGNED pointer to Reviewer
> 	When fedora-review? or fedora-review-
> 	Use NEEDINFO to request owner to fix something.

Again: shouldn't fedora-review- be a rather permanent state? I admit
that I don't entirely understand the purpose of those flags, whatever they
are.

> 3) State 3: Approved
> ASSIGNED pointer to Owner
> 	ONLY when fedora-review+

Leave assigned to reviewer, please. It's illogical to change it.

> Benefits
> ========
> - No bouncing back and forth between reviewer and owner.
> - http://rubenkerkhof.com/review
> This page continues to show under "Assigned" who is acting.
> - frontpage.cgi shows both ASSIGNED and NEEDINFO for appropriate actors 
> both during and after the review.

Good!

Regards,
R.

-- 
Fedora Extras contributor  http://fedoraproject.org/wiki/DominikMierzejewski
Livna contributor http://rpm.livna.org MPlayer developer http://mplayerhq.hu
"Faith manages."
        -- Delenn to Lennier in Babylon 5:"Confessions and Lamentations"

--
Fedora-maintainers mailing list
Fedora-maintainers@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-maintainers

--
Fedora-maintainers-readonly mailing list
Fedora-maintainers-readonly@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-maintainers-readonly

[Index of Archives]     [Fedora Users]     [Fedora Development]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux