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.

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

incorporate minimalism feedback on filters page #70366

Merged
merged 1 commit into from Jul 1, 2020

Conversation

acozine
Copy link
Contributor

@acozine acozine commented Jun 29, 2020

SUMMARY

Incorporates feedback from the Minimalism session on June 26, 2020. Thanks to @cbudz, @sayee-jadhav, @IngridT1, @sbream, Jennifer Ciroli, Lorena Bolanos Abarca, Bob Johnson, and Matt Werner. This PR is the next step. There's more that can be improved here, but each iteration is a step forward.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

docs.ansible.com
filters

@ansibot ansibot added affects_2.11 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. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Jun 29, 2020
@acozine
Copy link
Contributor Author

acozine commented Jun 29, 2020

@bobjohnsrh thanks again!

@acozine acozine removed the needs_triage Needs a first human triage before being processed. label Jun 29, 2020
@samccann samccann merged commit c89f3cd into ansible:devel Jul 1, 2020
samccann pushed a commit to samccann/ansible that referenced this pull request Jul 2, 2020
Co-authored-by: Alicia Cozine <acozine@users.noreply.github.com>
(cherry picked from commit c89f3cd)
@samccann samccann mentioned this pull request Jul 2, 2020
acozine added a commit that referenced this pull request Jul 8, 2020
* doc: avoid mix of single and double quotes (#70115)

Avoid mix of single and double quotes in the `ternary`, this way
we can copy/past the example without any surprise.

(cherry picked from commit b491f77)

* document FQCN for M()  and :seealso: in DOCUMENTATION blocks (#70245)

* document FQCN for M() in DOCUMENTATION blocks

* add note about c

(cherry picked from commit 83f6e48)

* Fix bullet points in intro_getting_started.rst. (#70365)

The layout was jumbled due to issues with whitespace.

(cherry picked from commit dc6f4b6)

* Add steps for how to create changelog.rst for a collection (#70262)

* Update docs/docsite/rst/dev_guide/developing_collections.rst
* add steps to create changelogs, add sentence about not using the tool
* add note for rerunning the command

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit 5a28b2b)

* ansible-doc: avoid problems with YAML anchors when formatting man page (#70045)

* Avoid problems with YAML anchors when formatting man page.

* Add changelog.

(cherry picked from commit 5e4f708)

* Minor grammatical fix (#70405)

'you' -> 'your'

(cherry picked from commit a1ac595)

* incorporate minimalism feedback on filters page (#70366)

Co-authored-by: Alicia Cozine <acozine@users.noreply.github.com>
(cherry picked from commit c89f3cd)

* more correct info about role main.yml (#70326)

fixes #40496

(cherry picked from commit 5d3d097)

* Fix a small typo in cache plugin description @ `config/base.yml`

PR #70420

(cherry picked from commit 626df08)

* with_sequence: example using vars (#69369)

Added an example for using vars in with_sequence.

Fixes: #68836

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
(cherry picked from commit 5709173)

* Update pull.py (#70393)

(cherry picked from commit 46ad3c1)

* Update playbooks.rst (#70317)

(cherry picked from commit 7c90a2d)

* Add documentation for ipaddr filters  (#70343)

(cherry picked from commit 9eb904e)

* update platform table with links to collections (#70373)

(cherry picked from commit aa59c23)

* Add description of collections and become_exe keywords (#68055)

* Add description of collections keyword
* Update based on feedback.
- Add link to become plugins.
- Add note about how the collections keyword works with roles.

(cherry picked from commit 5833af9)

Co-authored-by: Gonéri Le Bouder <goneri@lebouder.net>
Co-authored-by: Mark Sanders <ziplokk.mark.sanders@gmail.com>
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Sir Mobus Gochfulshigan Dorphin Esquire XXIII <celestialtuba@gmail.com>
Co-authored-by: Alicia Cozine <879121+acozine@users.noreply.github.com>
Co-authored-by: Brian Coca <bcoca@users.noreply.github.com>
Co-authored-by: Michael Scherer <mscherer@users.noreply.github.com>
Co-authored-by: Abhijeet Kasurde <akasurde@redhat.com>
Co-authored-by: Ethan <smithe2413@gmail.com>
Co-authored-by: jafiala <56597272+jafiala@users.noreply.github.com>
Co-authored-by: Baptiste Mille-Mathias <baptiste.millemathias@gmail.com>
Co-authored-by: Sam Doran <sdoran@redhat.com>
@ansible ansible locked and limited conversation to collaborators Jul 29, 2020
@acozine acozine deleted the minimalism_feedback branch August 3, 2020 18:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.11 core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants