Allow multi-line banners #123
Conversation
Currently a multi-line banner will cause the YAML file to be invalid. This change ensures a valid YAML format without changing behavior. Signed-off-by: tomi-bigpi <[email protected]>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Congrats on opening your first pull request and thank you for taking the time to help improve Ansible-Lockdown!
Please join in the conversation happening on the Discord Server as well.
|
hi @tomi-bigpi Thank you for raising this PR and helping to improve ansible-lockdown. The change makes sense, just like to understand why the indent has been added to line? We are trying to ensure all code is consitent where possible without exceptions. Is there something taht we can do to assist here? many thanks uk-bolly |
|
Hi @uk-bolly the indent is there to ensure the multi-line YAML is valid when using the References: |
Overall Review of Changes:
Currently a multi-line banner will cause the YAML file to be invalid. This change ensures a valid YAML format without changing behavior.
Issue Fixes:
Haven't opened an issue.
Enhancements:
Please list any enhancements/features that are not open issue tickets
How has this been tested?:
Manually tested with multi-line banner text.