On 10/19/22 04:33, Maxwell G wrote:
Hi Karolina,
Thank you for working on this! I have also found Zuul useful, but I
don't always remember to enable it for my packages. I have two
suggestions 1) It looks like some other lines were rearranged as part
of this change. Would it be possible to make that a separate change?
Hi Maxwell,
There are two commits in the PR
- one that only sorts the existing entries:
https://pagure.io/fork/ksurma/fedora-project-config/c/e8a9bfea2c084235bd8...
- one that only adds the new packages:
https://pagure.io/fork/ksurma/fedora-project-config/c/4b7c2f710393d18f385...
- that's pretty much the one of interest.
I was hoping this would make the diff clearer, but I must admit, for
over 700 packages it's not that visible. Which leads me to point 2. :)
2)
I would include a list of affected maintainers (this can be
generated
with [1]) and perhaps CC devel@ for increased visibility.
[1]:
https://pagure.io/fedora-misc-package-utilities/blob/master/f/find-packag...
That's a good idea - I will send the e-mail shortly.
--
Best,
Maxwell G (@gotmax23)
Pronouns: He/Him/His
--
Karolina Surma (she/her/hers)
Software Engineer
Python Maintenance Team, Red Hat