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
Module DOCUMENTATION should match argspec #34895
Conversation
Large update of many modules so that DOCUMENTATION option name and aliases match those defined in the argspec. Issues identified by ansible#34809 In addition to many typos and missing aliases, the following notable changes were made: * Create module_docs_fragments/url.py for `url_argument_spec` * dellos*_command shouldn't have ever had `waitfor` (was incorrectly copied) * ce_aaa_server_host.py `s/raduis_server_type/radius_server_type/g` * Junos_lldp enable should be part of `state`.
bot_status |
ComponentsMetadatawaiting_on: gundalow |
''' | ||
- aws | ||
- ec2 | ||
''' |
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.
Is this extra space accidental?
''' | ||
- aws | ||
- ec2 | ||
''' |
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.
Extra space
''' | ||
- aws | ||
- ec2 | ||
''' |
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.
Extra space
@@ -35,6 +35,7 @@ | |||
id: | |||
description: | |||
- Numeric, the droplet id you want to operate on. | |||
aliases: ['droplet_id'] |
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.
over indented
@@ -41,6 +41,7 @@ | |||
- Corresponds to the C(--no-recommends) option for I(urpmi). | |||
type: bool | |||
default: 'yes' | |||
aliases@ ['no-recommends'] |
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.
s/@/:/
In addition to my comments, from the last build (ignoring version_added comments which are invalid):
|
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.
Looks good, aside from what @sivel already noted.
Fixed all the issues |
SUMMARY
Large update of many modules so that DOCUMENTATION option name and
aliases match those defined in the argspec.
Issues identified by #34809
In addition to many typos and missing aliases, the following notable
changes were made:
module_docs_fragments/url.py
forurl_argument_spec
*
dellos*_command
shouldn't have ever hadwaitfor
(incorrectly copied deprecated aliases)ce_aaa_server_host.py
s/raduis_server_type/radius_server_type/g
junos_lldp
enable should be part ofstate
.ISSUE TYPE