Skip to content

ansible-lint - use changed_when even if using conditional#143

Merged
richm merged 1 commit intolinux-system-roles:mainfrom
richm:more-ansible-lint
Apr 8, 2023
Merged

ansible-lint - use changed_when even if using conditional#143
richm merged 1 commit intolinux-system-roles:mainfrom
richm:more-ansible-lint

Conversation

@richm
Copy link
Copy Markdown
Collaborator

@richm richm commented Apr 7, 2023

ansible-lint now requires the use of changed_when even if the
command already uses when

Signed-off-by: Rich Megginson rmeggins@redhat.com

ansible-lint now requires the use of `changed_when` even if the
command already uses `when`

Signed-off-by: Rich Megginson <rmeggins@redhat.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 7, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (424453a) 74.44% compared to head (69c4417) 74.44%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #143   +/-   ##
=======================================
  Coverage   74.44%   74.44%           
=======================================
  Files           1        1           
  Lines         450      450           
=======================================
  Hits          335      335           
  Misses        115      115           
Flag Coverage Δ
sanity ∅ <ø> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@richm
Copy link
Copy Markdown
Collaborator Author

richm commented Apr 7, 2023

[citest]

@richm richm merged commit 8e2cc81 into linux-system-roles:main Apr 8, 2023
@richm richm deleted the more-ansible-lint branch April 8, 2023 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant