-
Notifications
You must be signed in to change notification settings - Fork 681
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
Improve links to Learn Chef Rally #2476
Conversation
Signed-off-by: Thomas Petchel <[email protected]>
Signed-off-by: tpetchel <[email protected]>
9dcf786
to
208c97b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @tpetchel for that improvement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes overall look good. One wording modification I'd like to see for clarity before merge/release.
a.tutorials--link href="https://learn.chef.io/tracks/integrated-compliance/" | ||
i.focus ⪢ | ||
| Integrated Compliance with Chef | ||
div Continue your journey here if you're interested in using Chef to correct potential compliance and security issues. | ||
br | ||
p.clear Huge thanks to Annie Hedgpeth for creating all of these fantastic tutorials. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To make it clear which of the tutorials are Annie's, I suggest changing this line to:
"Huge thanks to Annie Hedgpeth for creating all of the following fantastic tutorials:"
@tpetchel Anything we can help to get this in? |
This is a good change as-is so I'll merge it and then put forth a quick one-liner change to update. |
We recently published some updates to Learn Chef Rally that we'd like to highlight.