Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

composer-on-rhel: Fix a couple of typos, hyperlinks, and a caution #18

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kyle-walker
Copy link

Fixing a reference to "Enterpise Linux" and changing the weldr.io
hyperlinks to point to the https:// URI. Since there is a reference to "not
using the system for other workloads", I also included a mention that it
would be best to deploy the utility in a VM.

Fixing a reference to "Enterpise Linux" and changing the weldr.io
hyperlinks to point to the https:// URI. Since there is a reference to "not
using the system for other workloads", I also included a mention that it
would be best to deploy the utility in a VM.
Adding a few changes to make the content cover the RHEL use-case. For RHEL
7.6, the software collections python36 revision can be used to install the
awscli utility. Also removed reference to specifically using Fedora 29 for
the process as it isn't necessary. Just a system that is RHEL 7.6 or above,
and Fedora 29 or above.
None of the content is inherently Fedora 29, and so these changes indicate
that the RHEL 7.6 release is within scope. In addition, changing the yum
repos entry to use the "yum-config-manager --add-repo=" operation as it is
somewhat less subject to typos.
None of the content is inherently Fedora 29, and so these changes indicate
that the RHEL 7.6 release is within scope. Set all hyperlinks to https.
None of the content is inherently Fedora 29, and so these changes indicate
that the RHEL 7.6 release is within scope. Set all hyperlinks to https.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant