Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

Use template module instead of copy #66

Merged
merged 1 commit into from
Sep 20, 2022
Merged

Conversation

tobiashuste
Copy link
Member

@tobiashuste tobiashuste commented Sep 20, 2022

According to the documentation using copy with variables can lead to unpredictable output. This is now also reflected in ansible-lint.
Compare with /~https://github.com/ansible/ansible-lint/blob/ac7284252d744318a446e3a4e9c788b923b767e7/src/ansiblelint/rules/template-instead-of-copy.md

The new rule is introduced in #65.

According to the documentation using copy with variables can lead to
unpredictable output. This is now also reflected in ansible-lint.
Comparr with /~https://github.com/ansible/ansible-lint/blob/ac7284252d744318a446e3a4e9c788b923b767e7/src/ansiblelint/rules/template-instead-of-copy.md
@tobiashuste tobiashuste force-pushed the template-instead-of-copy branch from 3ee6355 to d9d9cd8 Compare September 20, 2022 08:47
@tobiashuste tobiashuste self-assigned this Sep 20, 2022
@tobiashuste tobiashuste marked this pull request as ready for review September 20, 2022 08:49
@tobiashuste tobiashuste added the bug Something isn't working label Sep 20, 2022
@tobiashuste tobiashuste requested a review from Normo September 20, 2022 08:54
Copy link
Member

@Normo Normo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @tobiashuste!

@tobiashuste tobiashuste merged commit 70a758d into main Sep 20, 2022
@tobiashuste tobiashuste deleted the template-instead-of-copy branch September 20, 2022 09:05
@tobiashuste tobiashuste added this to the 1.2.0 milestone Oct 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants