Re: not all that qualified...

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

 



On 05/31/2017 05:19 PM, Christopher wrote:
On Wed, May 31, 2017 at 6:44 PM David Muse <david.muse@xxxxxxxxxxxxxx <mailto:david.muse@xxxxxxxxxxxxxx>> wrote:

    Hello all,

    I have a fairly large package that needs review:
    https://bugzilla.redhat.com/show_bug.cgi?id=1415612

    and I've been offered a few review swaps for it, but the trouble is, I
    don't really feel well qualified to review packages yet.  I only
    maintain one other package (a pre-requisite for that one above) and it's
    super simple.

    Should I just jump in anyway?  How is this kind of thing usually
    resolved?

At the risk of turning this into a "me too" thread, I want to thank you for asking this question. I'm in the same situation. I'm not comfortable reviewing packages yet. I'm sure there are others as well. So, on behalf of all us in this same situation, thanks for asking. I look forward to the advice that (hopefully) follows, from those more experienced.

Sounds like a Catch-22: you can't get experience without reviewing packages, but you don't want to review until you have more experience.

However, this is exactly why we have the sponsor system in place. I've found it a good system that the sponsor should ask for a few informal reviews from their sponsoree candidates, and the sponsor then does the formal review, checking if the sponsoree missed anything.

Reviewing is nowadays a much simpler task than what it used to be, thanks to the automated fedora-review process. It handles a lot of things for you, but you still do have to go through some checklists by hand.

I would suggest you do an informal review, and ask your sponsor, or people on the list to check it for you.
--
Susi Lehtola
Fedora Project Contributor
jussilehtola@xxxxxxxxxxxxxxxxx
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [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