-
Notifications
You must be signed in to change notification settings - Fork 330
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 Docs In azure_rm_virtualnetwork
#1203
Improve Docs In azure_rm_virtualnetwork
#1203
Conversation
@@ -218,7 +222,7 @@ def __init__(self): | |||
state=dict(type='str', default='present', choices=['present', 'absent']), | |||
location=dict(type='str'), | |||
address_prefixes_cidr=dict(type='list', aliases=['address_prefixes']), |
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.
address_prefixes_cidr=dict(type='list', aliases=['address_prefixes']), | |
address_prefixes_cidr=dict(type='list', elements='str', aliases=['address_prefixes']), |
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.
@Fred-sun I've updated the code to reflect this change. Thank you for the comment!
@@ -218,7 +222,7 @@ def __init__(self): | |||
state=dict(type='str', default='present', choices=['present', 'absent']), | |||
location=dict(type='str'), | |||
address_prefixes_cidr=dict(type='list', aliases=['address_prefixes']), | |||
dns_servers=dict(type='list',), | |||
dns_servers=dict(type='list'), |
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.
dns_servers=dict(type='list'), | |
dns_servers=dict(type='list', elements='str'), |
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.
@Fred-sun I've updated the code to reflect this change. Thank you for the comment!
* The DNS Servers should be a list of strings * Remove the trailing comma in the `dict()` call for `dns_servers` * The boolean values should default to a boolean value in the docs * Clean up excess quotation in some of the examples
* Incorporate PR feedback from @Fred-sun correctly indicated that I didn't apply the elements parameter to the argument spec for `address_prefixes_cidr` and `dns_servers` * This helps Ansible validate the list elements are the correct type for the 2 affected module parameters
7e1ef70
to
d2d4edf
Compare
@timway Please delete the corresponding line in the following file
|
b6432e2
to
d2d4edf
Compare
@timway Please update the corresponding Ignore*.txt, because we added elements in document and args, which should be deleted (ignore line)accordingly! Thank you very much! |
* As @Fred-sun pointed out a number of the sanity tests no longer need to ignore a failure condition with this PR
kindly ping! |
@Fred-sun I believe I addressed the ignores in the most recent commit. |
@timway Sorry, it has been added, I will push forward the merge. |
@xuzhang3 ready_for_review |
SUMMARY
dict()
call fordns_servers
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION