fedora-docs / package-maintainer-docs

#26 Refer to FESCo package review policy
Merged by oturpe. Opened by oturpe.
fedora-docs/ oturpe/package-maintainer-docs update-review-pages  into  master

Download 26.patch

Recently, the Package Review Policy was added to FESCo docs,
mostly based on review pages of this repo.
Add reference to the FESCo policy
so that it is clear where need to do reviews comes from.
Also remove some content that does not have to be duplicated here
since it is already available at the FESCo policy.

Also some minor fixes to Packege Review Policy
in a separate commit.

Relates to #21

@ankursinha Could you take a look at this pull request
and #30, #32 as well?
There are not a lot of us yet,
but I would still like to move away
from just committing stuff
to a review process.

Sure thing, assigning them all to myself now. Should hopefully have them reviewed before the end of the day.

Metadata Update from @ankursinha:
- Request assigned

This is also covered in the FESCo docs from the looks of it here: https://docs.fedoraproject.org/en-US/fesco/Package_review_policy/#what (paragraph at the end). So perhaps it needs to be linked in this section too, just for clarity.

I think this command is wrong. It needs to be fedpkg request-repo <name of package> <bugid>, no? Only if you're working in a SCM type git repo that's named after the package maybe fedpkg will pick it up, but I don't think newcomers start from such SCM type repos?

Apart from these bits, looks very good. Please feel free to correct and merge :) :thumbsup:

rebased onto 306e0732e9a1f2afdab163dc16a9f10438599460

rebased onto f37c55c8922c41a9961a1383ee2f8562667f3246

Pull-Request has been merged by oturpe

Thank you for the review!
I fixed the issues you listed,
also still some small formatting improvements.
Merged.
Metadata
ankursinha
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.