Skip to content

docs: fix example json in template.md #629

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

d-Rickyy-b
Copy link

Adds missing bracket.
Escapes quotes within a string.
Reformats whole example.

fixes #627

Adds missing bracket.
Escapes quotes within a string.
Reformats whole example.

fixes adnanh#627
"secret": "{{ getenv "XXXTEST_SECRET" | js }}",
"parameter":
{
"secret": "{{ getenv \"XXXTEST_SECRET\" | js }}",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Escaping these quotes is not accurate here. This is a Go template, not a JSON file.

Copy link
Author

Choose a reason for hiding this comment

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

True, I did not read the template.md file properly when I created this PR. The missing [ is the only actual issue.

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

Successfully merging this pull request may close these issues.

There is a syntax error in your templates file. Please fix for newer webhook users like myself.
2 participants