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
Correct Errors in Documentation #50507
Conversation
According to the module, "compose" is not required, but the docs said it was - removed the "required" tag. Additionally, the example for removing a stack did not include the required "name" option (this method is why "compose" is not required. +label: docsite_pr
Corrected issue to my "correction" on the description.
LGTM |
@unrinfosec thanks for improving the Ansible docs! Our guidelines call for removing |
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.
LGTM
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.
shipit
Thanks for this PR, @unrinfosec!
@sivel @danihodovic @acozine @DBendit thanks for your reviews and comments! I'll create a backport PR for stable-2.7. |
Ok, that wasn't necessary, we just added this module for 2.8... ;-) |
* Correct Errors in Documentation According to the module, "compose" is not required, but the docs said it was - removed the "required" tag. Additionally, the example for removing a stack did not include the required "name" option (this method is why "compose" is not required. +label: docsite_pr * Removed change to description Corrected issue to my "correction" on the description. * Removed required false * Removed all instances of required: false.
According to the module, "compose" is not required, but the docs said it was - removed the "required" tag. Additionally, the example for removing a stack did not include the required "name" option (this method is why "compose" is not required.
+label: docsite_pr
SUMMARY
There were some errors in the documentation that did not match what the module required.
ISSUE TYPE
COMPONENT NAME
docker_stack.py
ADDITIONAL INFORMATION
Just minor changed to documentation.