-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Body fails when it contains double quotes. #13
Comments
Hey, it should be fixed by 5264f5b, and I'll tag it as |
Hey, now it fails when the body contains a backslash. The python dependency is the downside. Actually, could you use jq itself? |
Running a quick test in my test repo produced this release, looks good to me. |
Uhm, I think it works as intended, but not for me. It has been a while or I'm beeing stupid. I'd like to do something like
But BODY only contains the first line. How would I go about producing a multiline body like this? |
Could you try with: body: >
${{ env.BODY }} and body: ${{ env.BODY }} Also, I'd avoid using echo "set-env name=BODY::$(printf 'Hello!\nWorld!\n')" or text="Hello!
World!"
echo "set-env name=BODY::$text" |
You were right about double quotes, of course. I figured out a workaround now from actions/create-release#11. The trick is
They've also added a way to directly take the body from a file, which I find quite handy: actions/create-release#50 |
Ok, closing the issue again, as the problem seems to be resolved. |
Like #10 (comment), probably need to replace
"
with\"
. Thesed
filter could probably be moved inline totoJsonOrNull
to make sure it covers other fields (really just release name).I know this action is semi-deprecated but I am also stuck on this until the GitHub provided one handles some of the same use cases as your fantastic action.
I'll try to open a PR to fix this but I've worked around it on my end already so unless it comes up again, I may forget, so I thought I'd document it in an issue for the time being.
The text was updated successfully, but these errors were encountered: