Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

doc: add escaping special characters note #328

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jmtaylor90
Copy link

Signed-off-by: jason taylor jtfas90@gmail.com

Make sure these boxes are signed before submitting your Pull Request
-- thank you.

Link to redmine ticket:

Describe changes:

  • Add note about need to escape special characters in configuration files if they need to be evaluated literally

Signed-off-by: jason taylor <jtfas90@gmail.com>
@jmtaylor90 jmtaylor90 requested a review from jasonish as a code owner February 6, 2023 23:39
@jmtaylor90
Copy link
Author

jmtaylor90 commented Apr 24, 2024

Is this something that you all are interested in reviewing/merging or shall I close it? @jasonish

@jasonish
Copy link
Member

Is this something that you all are interested in reviewing/merging or shall I close it? @jasonish

Its worthy of documenting. I wonder if it should be more than a note with an example or 2.

@jmtaylor90
Copy link
Author

Is this something that you all are interested in reviewing/merging or shall I close it? @jasonish

Its worthy of documenting. I wonder if it should be more than a note with an example or 2.

Okay, will update with some examples. Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants