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 double-quote handling in release creation workflow #536

Merged
merged 2 commits into from
Jan 23, 2025

Conversation

Yang-33
Copy link
Contributor

@Yang-33 Yang-33 commented Jan 21, 2025

This PR escapes the release title’s double quotes and provides the release notes via a file, preventing shell interpretation errors in the gh release create command.

Current workflow fails when
(1) release title contains ": https://github.com/line/line-bot-sdk-java/actions/runs/12886931493/job/35928566569#step:6:3
or (2) PR title contains " (e.g. on reverting change, title is revert "original title")

(same as line/line-bot-sdk-java#1536)

@Yang-33 Yang-33 changed the title Output created release note once before passing gh command Output created release note once before passing it to gh command Jan 21, 2025
@Yang-33 Yang-33 changed the title Output created release note once before passing it to gh command Output created release note to temp file once before passing it to gh command Jan 21, 2025
@Yang-33 Yang-33 closed this Jan 21, 2025
@Yang-33 Yang-33 deleted the Yang-33-patch-1 branch January 21, 2025 12:45
@Yang-33 Yang-33 restored the Yang-33-patch-1 branch January 21, 2025 13:07
@Yang-33 Yang-33 reopened this Jan 21, 2025
@Yang-33 Yang-33 changed the title Output created release note to temp file once before passing it to gh command Fix double-quote handling in release creation workflow Jan 21, 2025
@Yang-33 Yang-33 merged commit e7b0091 into master Jan 23, 2025
4 checks passed
@Yang-33 Yang-33 deleted the Yang-33-patch-1 branch January 23, 2025 02:44
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.

2 participants