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

Update YAMLSyntax.rst #47491

Merged
merged 2 commits into from
Oct 25, 2018
Merged

Update YAMLSyntax.rst #47491

merged 2 commits into from
Oct 25, 2018

Conversation

d3nisko
Copy link
Contributor

@d3nisko d3nisko commented Oct 23, 2018

The provided example for list is not accurate. In such way we will have a dictionary (not list) with one key/pair value. Dict will have 'fruits' key and the value will be list of fruits. Therefore we have to remove "fruits:" statement to have just one list of fruits.
+label: docsite_pr

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME
ANSIBLE VERSION
ADDITIONAL INFORMATION

@d3nisko
Update YAMLSyntax.rst
b502100 
<!--- Your description here -->
The provided example for list is not accurate. In such way we will have a dictionary with one key/pair value. Dict will have 'fruit' key and the value will be list of fruits. Therefore we have to remove "fruit:" statement to have just one list of fruits. 
+label: docsite_pr
@ansibot
Copy link
Contributor

ansibot commented Oct 23, 2018

Hi @d3nisko, thank you for submitting this pull-request!

click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Oct 23, 2018

cc @acozine
click here for bot help

@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Oct 23, 2018
@ansibot ansibot removed the needs_triage Needs a first human triage before being processed. label Oct 25, 2018
@samccann samccann merged commit 6c2e4c1 into ansible:devel Oct 25, 2018
@samccann
Copy link
Contributor

Thanks @d3nisko for updating the Ansible documentation!

@dagwieers dagwieers added the docsite_pr This PR is created from documentation using the "Edit on GitHub" link. label Nov 17, 2018
Tomorrow9 pushed a commit to Tomorrow9/ansible that referenced this pull request Dec 4, 2018
@d3nisko @Tomorrow9
Update YAMLSyntax.rst (ansible#47491)
e73c221 
* Update YAMLSyntax.rst to correct syntax of a list vs dict.
@ansible ansible locked and limited conversation to collaborators Jul 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@samccann samccann samccann approved these changes

Assignees
No one assigned
Labels
affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. docsite_pr This PR is created from documentation using the "Edit on GitHub" link. new_contributor This PR is the first contribution by a new community member. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@d3nisko @ansibot @samccann @dagwieers