Il 06/02/24 14:34, Zbigniew Jędrzejewski-Szmek ha scritto:
On Mon, Jan 29, 2024 at 06:47:21PM +0000, Mattia Verga via devel
wrote:
> That said, I'd like to make a request and maybe make all reviewers aware
> of a feature which was implemented some time ago. I've noticed many
> reviewers change the ticket status from ASSIGNED to POST when they flag
> the package as approved: I'd like to request to not do that.
The reason for POST is that this allows the reviews that are "done"
from the reviewer's side to be easily distinguished in a bugzilla
listing. E.g. my reviews are [1], and then I only need to care about
those with ASSIGNED.
If we're not supposed to use POST, then please provide this
functionality in some different form. (I think the suggestion
from the other part of thread to use RELEASE_PENDING is better.)
Sure, the 'POST' usage has been inherited by the old review ticket
status webpages. When I rewrite the code I used the same statuses, but I
agree that it may have much more sense to use 'RELEASE_PENDING'. If
there isn't any objection e.g. disrupting someone's else tool workflow,
I can easily change that.
BTW, the ticket status is not changed automatically upon repository
creation because I now realize there was an error in the code I
submitted to toddlers. There is now a PR to fix that [1]
Mattia
[1]
https://pagure.io/fedora-infra/toddlers/pull-request/184