Skip to content

Conversation

@garyjcarroll
Copy link
Contributor

@garyjcarroll garyjcarroll commented Jan 16, 2025

Before submitting a pull request for a new Learning Path, please review Create a Learning Path

  • I have reviewed Create a Learning Path

Please do not include any confidential information in your contribution. This includes confidential microarchitecture details and unannounced product information. No AI tool can be used to generate either content or code when creating a learning path or install guide.

  • I have checked my contribution for confidential information

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the Creative Commons Attribution 4.0 International License.

This point was raised in feedback from AWS:

"source is used several times in the install instructions, it’s not posix so
will fail if anyone try to copy/paste the instructions into a script executed
with /bin/sh which points to dash and is strictly posix on ubuntu"
Want to make clear that AL2023 has its own rpm repo.
Copy link
Collaborator

@madeline-underwood madeline-underwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

“Setup” should be “set up” when used as a verb.

@garyjcarroll
Copy link
Contributor Author

Thanks for spotting. And excuse the "setup" typo in the commit message :)

@garyjcarroll
Copy link
Contributor Author

@madeline-underwood I could use some advice on how to resolve the merge conflict that has appeared on this PR. The documentation link Create a Learning Path is returning a 404...

@madeline-underwood
Copy link
Collaborator

@madeline-underwood I could use some advice on how to resolve the merge conflict that has appeared on this PR. The documentation link Create a Learning Path is returning a 404...

Hi Gary, let's wait for some of the merges to be made by the project maintainers and we will see if this resolves any of these conflicts. If not, I'll see what I can do to help. Many thanks again for your contribution.

@garyjcarroll
Copy link
Contributor Author

Changes made with new additions in separate PR

#1557

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.

2 participants