Re: Package review ticket status change after approval

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

 



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

--
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux