On Fri, 2 Dec 2022 at 18:21, Jakub Kadlcik <jkadl...@redhat.com> wrote: > > Hello folks, > > for a couple of years now, I've been interested in the Fedora package > review process. Our queue is in hundreds of waiting packages for as > long as I can remember. I believe the situation can be improved. > > Since summer, I did ~40 package reviews to get a better grasp of the > situation and realized what I want to do. > > I started working on a service [2] that listens to fedora-messaging > and for every new RHBZ review ticket or a new comment with updated > packages, it submits a build in Copr. Thanks to this [1] feature, Copr > automatically runs the fedora-review tool and generates the > review.txt file. Once the build is finished, my service gets the > message and generates a helpful comment (so far only to STDOUT). > > **Unless there is general disapproval, I am planning to let it post > the comments to Bugzilla.**
Yes, please! This seems like a **very** helpful service to me. Thanks for this! :) Iñaki > So far the benefit is limited but, it still can immediately tell the > contributor that their package is broken and fails to build, and also > saves a reviewer the time of running the fedora-review tool > manually. However, I also implemented support for the fedora-review > tool to generate a JSON output next to the standard review.txt. The PR > is still pending [2], please review it if you can. Once this is > released, I can parse its contents and generate much helpful comments > for the contributors. > > The end goal is to let contributors go back and forth against the CI > to fix the most obvious mistakes and then let the reviewers take > only the final look. > > Hopefully, it will be a better experience for everyone. > > > [1] http://frostyx.cz/posts/running-fedora-review-after-copr-build > [2] https://github.com/FrostyX/fedora-review-service > [3] https://pagure.io/FedoraReview/pull-request/463 > > Jakub > _______________________________________________ > devel mailing list -- devel@lists.fedoraproject.org > To unsubscribe send an email to devel-le...@lists.fedoraproject.org > 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@lists.fedoraproject.org > Do not reply to spam, report it: > https://pagure.io/fedora-infrastructure/new_issue -- Iñaki Úcar _______________________________________________ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org 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@lists.fedoraproject.org Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
