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
removes broken link to old module name #39249
Conversation
@@ -21,7 +21,7 @@ | |||
description: | |||
- makes a computer speak! Amuse your friends, annoy your coworkers! | |||
notes: | |||
- In 2.5, this module has been renamed from M(osx_say) into M(say). | |||
- In 2.5, this module has been renamed from ``osx_say`` to M(say). |
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.
osx_say is still there as a symlink, unsure if this is the correct solution as this will be the case with any aliased module.
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.
Please change to:
from C(osx_say) to M(say)
No need to have a link from say to osx_say, as you'd end up on the same page
3f32296
to
95ad31b
Compare
@gundalow fixed, thanks |
SUMMARY
As part of making all rST warnings fatal on Shippable, we are eliminating warnings from the docs build. This PR addresses one of these:
say
module contains a link to its old name, triggering aWARNING: undefined label
error. This PR retains the text but removes the link.ISSUE TYPE
COMPONENT NAME
say module
ANSIBLE VERSION
2.5