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

Windows: Small fix to Add-Warning and Add-DeprecationWarning #23520

Merged
merged 1 commit into from
May 9, 2017

Conversation

dagwieers
Copy link
Contributor

SUMMARY

Simplified the logic, and fix the issue of having a nested list as
entry.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

powershell.ps1

ANSIBLE VERSION

v2.3

@ansibot ansibot added affects_2.4 This issue/PR affects Ansible v2.4 bugfix_pull_request c:module_utils/ needs_triage Needs a first human triage before being processed. windows Windows community labels Apr 12, 2017
@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Apr 12, 2017
@nitzmahone nitzmahone removed the needs_triage Needs a first human triage before being processed. label Apr 13, 2017
@ansibot ansibot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels May 9, 2017
@ansibot ansibot removed the needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html label May 9, 2017
@dagwieers
Copy link
Contributor Author

@nitzmahone Unrelated issues with async_wrapper on Windows ?

Simplified the logic, and fix the issue of having a nested list as
entry.

Also indicate that we deliberately want to retain $null values
(and i.e. do not want them to be turned into a boolean).
@nitzmahone nitzmahone merged commit bc806ff into ansible:devel May 9, 2017
dpiotrowski pushed a commit to dpiotrowski/ansible that referenced this pull request May 9, 2017
…#23520)

Simplified the logic, and fix the issue of having a nested list as
entry.

Also indicate that we deliberately want to retain $null values
(and i.e. do not want them to be turned into a boolean).
KKoukiou pushed a commit to KKoukiou/ansible that referenced this pull request May 22, 2017
…#23520)

Simplified the logic, and fix the issue of having a nested list as
entry.

Also indicate that we deliberately want to retain $null values
(and i.e. do not want them to be turned into a boolean).
@ansibot ansibot added bug This issue/PR relates to a bug. and removed bugfix_pull_request labels Mar 6, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.4 This issue/PR affects Ansible v2.4 bug This issue/PR relates to a bug. c:module_utils/ needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. windows Windows community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants