Skip to content
This repository was archived by the owner on Nov 19, 2025. It is now read-only.

use comment filter to support multiline ansible management message#5

Open
FrankGiesecke wants to merge 11 commits intoarillso:masterfrom
FrankGiesecke:fix-ansible-comment
Open

use comment filter to support multiline ansible management message#5
FrankGiesecke wants to merge 11 commits intoarillso:masterfrom
FrankGiesecke:fix-ansible-comment

Conversation

@FrankGiesecke
Copy link
Copy Markdown

The current usage results in invalid configuration files if the variable ansible_managed contains a multiline message.

It's better to use the comment filter, which supports multiline management messages.

Copy link
Copy Markdown

@thalunil thalunil left a comment

Choose a reason for hiding this comment

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

looks good!

@FrankGiesecke
Copy link
Copy Markdown
Author

Please, can some one fix the build errors in travis CI.

@thalunil
Copy link
Copy Markdown

Please, can some one fix the build errors in travis CI.

See #6

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants