Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Sanity test / doc-default-does-not-match-spec fixups #320

Merged
merged 2 commits into from
Dec 7, 2020
Merged

Sanity test / doc-default-does-not-match-spec fixups #320

merged 2 commits into from
Dec 7, 2020

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Dec 6, 2020 •
edited
Loading

SUMMARY

Daily run picked up the following:

plugins/modules/aws_api_gateway.py:0:0: doc-default-does-not-match-spec: Argument 'tracing_enabled' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/aws_application_scaling_policy.py:0:0: doc-default-does-not-match-spec: Argument 'override_task_capacity' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/aws_direct_connect_connection.py:0:0: doc-default-does-not-match-spec: Argument 'forced_update' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/aws_direct_connect_link_aggregation_group.py:0:0: doc-default-does-not-match-spec: Argument 'delete_with_disassociation' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/aws_direct_connect_link_aggregation_group.py:0:0: doc-default-does-not-match-spec: Argument 'force_delete' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/aws_direct_connect_link_aggregation_group.py:0:0: doc-default-does-not-match-spec: Argument 'wait' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/cloudfront_distribution.py:0:0: doc-default-does-not-match-spec: Argument 'enabled' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/cloudfront_distribution.py:0:0: doc-default-does-not-match-spec: Argument 'ipv6_enabled' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/ec2_ami_copy.py:0:0: doc-default-does-not-match-spec: Argument 'encrypted' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/ec2_vpc_vpn.py:0:0: doc-default-does-not-match-spec: Argument 'purge_routes' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/ecs_ecr.py:0:0: doc-default-does-not-match-spec: Argument 'purge_lifecycle_policy' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/ecs_ecr.py:0:0: doc-default-does-not-match-spec: Argument 'purge_policy' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/ecs_service.py:0:0: doc-default-does-not-match-spec: Argument 'force_new_deployment' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/ecs_taskdefinition.py:0:0: doc-default-does-not-match-spec: Argument 'force_create' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/elasticache.py:0:0: doc-default-does-not-match-spec: Argument 'hard_modify' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/elb_application_lb.py:0:0: doc-default-does-not-match-spec: Argument 'deletion_protection' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/elb_application_lb.py:0:0: doc-default-does-not-match-spec: Argument 'http2' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/elb_classic_lb.py:0:0: doc-default-does-not-match-spec: Argument 'cross_az_load_balancing' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/elb_network_lb.py:0:0: doc-default-does-not-match-spec: Argument 'cross_zone_load_balancing' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/elb_network_lb.py:0:0: doc-default-does-not-match-spec: Argument 'deletion_protection' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/elb_target.py:0:0: doc-default-does-not-match-spec: Argument 'deregister_unused' in argument_spec defines default as (False) but documentation defines default as (None)
plugins/modules/iam_cert.py:0:0: doc-default-does-not-match-spec: Argument 'dup_ok' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/route53.py:0:0: doc-default-does-not-match-spec: Argument 'alias' in argument_spec defines default as (None) but documentation defines default as (False)
plugins/modules/sqs_queue.py:0:0: doc-default-does-not-match-spec: Argument 'content_based_deduplication' in argument_spec defines default as (None) but documentation defines default as (False)

Some additional doc-style fixups.

Fallout from ansible/ansible#72699

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

plugins/modules/aws_api_gateway.py
plugins/modules/aws_application_scaling_policy.py
plugins/modules/aws_direct_connect_connection.py
plugins/modules/aws_direct_connect_link_aggregation_group.py
plugins/modules/cloudfront_distribution.py
plugins/modules/ec2_ami_copy.py
plugins/modules/ec2_vpc_vpn.py
plugins/modules/ecs_ecr.py
plugins/modules/ecs_service.py
plugins/modules/ecs_taskdefinition.py
plugins/modules/elasticache.py
plugins/modules/elb_application_lb.py
plugins/modules/elb_classic_lb.py
plugins/modules/elb_network_lb.py
plugins/modules/elb_target.py
plugins/modules/iam_cert.py
plugins/modules/route53.py
plugins/modules/sqs_queue.py

ADDITIONAL INFORMATION

@tremble tremble requested a review from felixfontein December 6, 2020 18:30
@tremble tremble changed the title [WIP] Sanity test / doc-default-does-not-match-spec fixups Sanity test / doc-default-does-not-match-spec fixups Dec 6, 2020
felixfontein
felixfontein reviewed Dec 7, 2020
View reviewed changes
@tremble tremble force-pushed the sanity/doc-default branch from 42bf6ed to fb63544 Compare December 7, 2020 06:35
@tremble tremble requested a review from felixfontein December 7, 2020 06:36
@tremble tremble merged commit b22f280 into ansible-collections:main Dec 7, 2020
@tremble
Copy link
Contributor Author

tremble commented Dec 7, 2020

Approved by Felix, tests passing. Merging.

@tremble tremble deleted the sanity/doc-default branch February 11, 2021 09:13
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 16, 2021
@tremble
[promoted]Sanity test / doc-default-does-not-match-spec fixups (ansib…
e0bf621 
…le-collections#320)

* Sanity test / doc-default-does-not-match-spec fixups
* General docs cleanup

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@b22f280
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
@tremble
Sanity test / doc-default-does-not-match-spec fixups (ansible-collect…
9bb99eb 
…ions#320)

* Sanity test / doc-default-does-not-match-spec fixups
* General docs cleanup
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
@tremble
Sanity test / doc-default-does-not-match-spec fixups (ansible-collect…
b1e03fd 
…ions#320)

* Sanity test / doc-default-does-not-match-spec fixups
* General docs cleanup
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
@tremble @danielcotton
Sanity test / doc-default-does-not-match-spec fixups (ansible-collect…
322d1e5 
…ions#320)

* Sanity test / doc-default-does-not-match-spec fixups
* General docs cleanup
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
@tremble @goneri
Sanity test / doc-default-does-not-match-spec fixups (ansible-collect…
e801557 
…ions#320)

* Sanity test / doc-default-does-not-match-spec fixups
* General docs cleanup

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@b22f280
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
@tremble @github-actions
Sanity test / doc-default-does-not-match-spec fixups (ansible-collect…
13eb44f 
…ions#320)

* Sanity test / doc-default-does-not-match-spec fixups
* General docs cleanup

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@b22f280
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@felixfontein felixfontein felixfontein approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tremble @felixfontein