Howdy Cody!

You definitely should be able to submit a PR without being in the packager group. That note you found was about write access, i.e. committing directly or merging PRs.

The fix itself is a good catch, but I actually think we should take a slightly different approach. Instead of just fixing the double alias in Packaging_Tutorial_2_GNU_Hello.adoc, I think we should move the quick-docs::creating-rpm-packages.adoc alias to Packaging_Tutorial_1_banner.adoc. That is based on the commit description that introduced the banner tutorial, which intends to make that the introductory point for users. I'll leave this open for comment from other docs maintainers for a bit to see if they agree or not.

After that is in place we should also look at updating the packaging guidelines to point directly to the new link instead of relying on the alias.

Hello! Anybody can definitely submit a pull request. The wording in the README does not exactly cover your case, but a) if you prefer to not have commit access, you can just say so, and we will not add you and b) access restriction is primarily an anti-spam measure, and I think this exchange has already established that your objective is to improve the docs, rather than spamming.

Note that there in an in-flight PR #155 tidying some other issues caused by addition or Banner tutorial. I will edit that one to:

1. Move creating-rpm-packages alias to the proper page, which is tutorial's table of contents page added in that PR.
2. Ensure that no alias is broken as described here

Updating the Quick Docs page is a good idea, the problem is the usual one: finding somebody who would bother.

That was my initial instinct, too. I saw the commit log mention why the GNU Hello program isn’t the best candidate for the stepping-stone tutorial. Then I saw the alias was moved to the renamed and realized it was a bug/oversight with the attribute usage. So, I opted to keep everything as-is instead of making a decision I have no right to make :).

Once I get confirmation, I’ll update and submit the PR accordingly.

You definitely should be able to submit a PR without being in the packager group. That note you found was about write access, i.e. committing directly or merging PRs.

Oh, I completely misinterpreted that. Thanks for the clarification.

EDIT

Oops, it looks like @oturpe replied the same time I did. Let me know if you want me to update this issue & submit a PR to fix this tiny edge case or if you want to keep everything in #155.

I added the fix into the existing pull requst. I am not completely sure how long it takes for the published page to get regenerated, but if you do not see it working in a couple of days, please let us know.

In case you find something else to fix or improve, please open another issue or pull request!

I can confirm that the changes in #155 are live. The link from the packaging guidelines is no longer broken, so this issue is resolved. Thanks for reporting it @codyrobertson.

For posterity, I also updated the link in the Package Guidelines to use the direct link instead of relying on the alias.

https://pagure.io/packaging-committee/c/92d0338f36f2b47d3807e5890e8f6d6185fffbf0?branch=master