-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
utilities: Clean up parameter types and add seealso #53063
Conversation
ca1aa2c
to
bbcdb58
Compare
This comment has been minimized.
This comment has been minimized.
d11ccf1
to
ec6634b
Compare
ec6634b
to
c4647c0
Compare
This comment has been minimized.
This comment has been minimized.
ddc737a
to
695ecc1
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.
Nice work on the seealso
d741ffb
to
fb01fa3
Compare
@gundalow Can we get this merged ? I had to rebase due to another conflict. |
@mattclay: I don't see a real cause, but I did find:
|
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.
Merge once CI is green
@dagwieers Recent changes to the ansible/test/integration/targets/assert/runme.sh Lines 25 to 26 in c0a4cf2
Those changes should not have been merged. Someone needs to update the test to work without requiring perl. |
@dagwieers I've fixed the |
SUMMARY
This PR includes:
seealso:
references to other modulesISSUE TYPE
COMPONENT NAME
utilities modules