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

Update intro_installation.rst #62994

Merged
merged 1 commit into from
Oct 3, 2019
Merged

Update intro_installation.rst #62994

merged 1 commit into from
Oct 3, 2019

Conversation

usrbinkat
Copy link
Contributor

@usrbinkat usrbinkat commented Oct 1, 2019

SUMMARY

Update wording & grammar correction

ISSUE TYPE
  • Docs Pull Request

+label: docsite_pr

SUMMARY

By adding module hyper-link references, the parentheses disclaimer may be removed and grammar corrections more easily made.

Visually modifying per the following:

Original: Ansible's "raw" module (for executing commands in a quick and dirty way) and the script module don't even need Python installed.

Proposed: Ansible's raw module, and the script module, do not depend on client side Python to run.

ISSUE TYPE
  • Docs Pull Request

##### SUMMARY
<!--- Update wording & grammar correction -->

Visually modifying per the following:
 - Ansible's "raw" module (for executing commands in a quick and dirty way) and the script module don't even need Python installed.
 + Ansible's raw module, and the script module, do not depend on client side Python to run.

By adding a raw module hyper-link reference, the parentheses disclaimer may be removed and grammar corrections more easily made.  

##### ISSUE TYPE
- Docs Pull Request

+label: docsite_pr
@ansibot
Copy link
Contributor

ansibot commented Oct 1, 2019

@ansibot ansibot added affects_2.10 This issue/PR affects Ansible v2.10 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. small_patch support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Oct 1, 2019
@dagwieers dagwieers added the docsite_pr This PR is created from documentation using the "Edit on GitHub" link. label Oct 1, 2019
@ansibot ansibot removed the needs_triage Needs a first human triage before being processed. label Oct 1, 2019
@samccann samccann merged commit f221495 into ansible:devel Oct 3, 2019
@samccann
Copy link
Contributor

samccann commented Oct 3, 2019

thanks @usrbinkat for the Ansible documentation fix!

acozine pushed a commit to acozine/ansible that referenced this pull request Oct 8, 2019
@acozine acozine mentioned this pull request Oct 8, 2019
acozine added a commit that referenced this pull request Oct 9, 2019
* Concise file module example description (#62932)
A file won't be created by the example, the execution would fail unless the file already exists as specified by the module options

(cherry picked from commit 87a6f83)

* Added suffix note to process_name_exact parameter (#62849)

(cherry picked from commit 992b81e)

* Fix broken link to powershell resources (#62845)
The previous link (https://docs.microsoft.com/en-us/powershell/dsc/builtinresource ) is a 404 . MS has a new link https://docs.microsoft.com/en-us/powershell/dsc/resources/resources  with the same content

(cherry picked from commit cae4255)

* Update playbooks_environment.rst (#62800)

(cherry picked from commit cea8a22)

* Update win_dsc.py (#62863)

(cherry picked from commit 798c5a5)

* Adjust documentation for yum clean to use best practices (#62898)
The yum module docs suggested using `shell: yum clean all`, which gives an `ansible-lint` error. Plus, you should use the `command` module unless you need more advanced features than just passing in an arbitrary command. Fixes that problem by using the `command` module in the example instead.

(cherry picked from commit 1e6fd6f)

* This should never have been "support:network" (#63068)

(cherry picked from commit 7ae4632)

* Add docs on hiding sensitive data in loops (#63096)

(cherry picked from commit 0311897)

* Doc updates to openssl_certificate and ecs_certificate (#63006)
* Update lib/ansible/modules/crypto/entrust/ecs_certificate.py

Co-Authored-By: Felix Fontein <felix@fontein.de>
(cherry picked from commit e55f46f)

* Update intro_installation.rst (#62994)

(cherry picked from commit f221495)

* Update xml.py (#63128)
Update example to remove escape characters in front of special characters as it is not needed to add explicitly anymore with latest ansible version

(cherry picked from commit 82a6f9d)
@ansible ansible locked and limited conversation to collaborators Nov 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.10 This issue/PR affects Ansible v2.10 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. small_patch 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

4 participants