Skip to content
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

fix(infra): escape git tag message #70

Merged
merged 1 commit into from
Jan 26, 2022

Conversation

miraclx
Copy link
Contributor

@miraclx miraclx commented Jan 26, 2022

Patches #67, just one last thing;

Ensures to embed the whole of the tag message in the release body. Not just the first line.

@miraclx miraclx force-pushed the automated-publishing-escape-git-body branch from 4a1a62f to dd05c7c Compare January 26, 2022 10:41
@matklad matklad merged commit 5c10694 into master Jan 26, 2022
@matklad matklad deleted the automated-publishing-escape-git-body branch January 26, 2022 10:57
matklad pushed a commit that referenced this pull request Jan 26, 2022
🔗 #70

Multi-line step outputs get truncated to the first line only, we have to manually escape the newline chars when outputting and runner would do the reverse when queried.

https://github.community/t/set-output-truncates-multiline-strings/16852/3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants