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

remove deprecated ansible.module_utils._text from documentation #73211

Merged
merged 1 commit into from
Apr 20, 2021

Conversation

schurzi
Copy link
Contributor

@schurzi schurzi commented Jan 13, 2021

SUMMARY

According to comment in ansible.module_utils._text it is deprecated and
should not be used. This is now reflected in the documentation.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

ansible.module_utils._text

ADDITIONAL INFORMATION

Once this is accepted I'm not sure on how to continue. We should also change all usages in the Ansible code. I could do that, if it is wanted.

According to comment in ansible.module_utils._text it is deprecated and
should not be used. This is now reflected in the documentation.
@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. docsite This issue/PR relates to the documentation website. 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 Jan 13, 2021
@bcoca bcoca removed the needs_triage Needs a first human triage before being processed. label Jan 14, 2021
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Jan 22, 2021
@ansibot ansibot added the docs_only All changes are to files within the docs/docsite/ directory label Mar 27, 2021
@samccann samccann added the pr_day Has been reviewed during a PR review Day label Apr 20, 2021
@acozine
Copy link
Contributor

acozine commented Apr 20, 2021

@samccann
Copy link
Contributor

rebuild_merge

@ansibot ansibot added shipit This PR is ready to be merged by Core and removed core_review In order to be merged, this PR must follow the core review workflow. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. labels Apr 20, 2021
@schurzi
Copy link
Contributor Author

schurzi commented Apr 20, 2021

how to continue with code changes:
[17:06:09] do a PR that updates at most 25 files
[17:06:48] you can post it here and in the ansible-devel channel

I will do that, when I find some time. single PRs per function/plugin seem to be the best approach

@samccann samccann merged commit 5e5bfa8 into ansible:devel Apr 20, 2021
@samccann
Copy link
Contributor

Thanks @schurzi for the Ansible docs fix!

acozine pushed a commit to acozine/ansible that referenced this pull request Apr 27, 2021
…ble#73211)

According to comment in ansible.module_utils._text it is deprecated and
should not be used. This is now reflected in the documentation.

(cherry picked from commit 5e5bfa8)
@acozine acozine mentioned this pull request Apr 27, 2021
acozine added a commit that referenced this pull request Apr 28, 2021
* Fix issue with version 3 in docs version list (#74089)

Previously would subsitute the "3" in "s3" instead of the version location in the URL

(cherry picked from commit 325ccf2)

* corrected epmhasis line (#74254)

need to be careful when adding lines as you create a different offset

(cherry picked from commit 7b39ee3)

* correct how to install Docker SDK for Python (#74288)

* docs: correct guide for the latest Docker SDK for Python
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Sandra McCann <samccann@redhat.com>
Co-authored-by: Daeseok Youn <daeseok.youn@navercorp.com>
Co-authored-by: Alicia Cozine <879121+acozine@users.noreply.github.com>

(cherry picked from commit 9369bd6)

* Provide results to examples (#73984)

(cherry picked from commit 99a2b5f)

* remove deprecated ansible.module_utils._text from documentation (#73211)

According to comment in ansible.module_utils._text it is deprecated and
should not be used. This is now reflected in the documentation.

(cherry picked from commit 5e5bfa8)

* Update playbooks_filters.rst (#74242)

##### SUMMARY
Make the `random` filter description more clear.

(cherry picked from commit 5f391a7)

* Docs: Fix k8s_config_resource_name YAML example (#74129)

The `name` key should be beneath `metadata`:

(cherry picked from commit c9c8459)

* Update lookup.rst (#73716)

Document that users must pass `allow_unsafe=True` as an option in the lookup to allow templating, with a note about security implications.

Co-authored-by: Alicia Cozine <879121+acozine@users.noreply.github.com>
Co-authored-by: Sandra McCann <samccann@redhat.com>
(cherry picked from commit c0cc574)

* vmware: Add a note about known issue (#73273)

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
Co-authored-by: Sandra McCann <samccann@redhat.com>
Co-authored-by: Alicia Cozine <879121+acozine@users.noreply.github.com>
(cherry picked from commit 6e56e72)

Co-authored-by: Scott Sinclair <252082+pwae@users.noreply.github.com>
Co-authored-by: Brian Coca <bcoca@users.noreply.github.com>
Co-authored-by: daeseokyoun <daeseok.youn@gmail.com>
Co-authored-by: Baptiste Mille-Mathias <baptiste.millemathias@gmail.com>
Co-authored-by: schurzi <github@drachen-server.de>
Co-authored-by: yuri <1969yuri1969@gmail.com>
Co-authored-by: ml <6209465+ml-@users.noreply.github.com>
Co-authored-by: sry9681 <sry9681@users.noreply.github.com>
Co-authored-by: Abhijeet Kasurde <akasurde@redhat.com>
@ansible ansible locked and limited conversation to collaborators May 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.11 docs_only All changes are to files within the docs/docsite/ directory docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. new_contributor This PR is the first contribution by a new community member. pr_day Has been reviewed during a PR review Day shipit This PR is ready to be merged by Core 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

6 participants