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

Fix double-quote handling in release creation workflow #1536

Merged
merged 2 commits into from
Jan 23, 2025
Merged

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")

@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 2df74be into master Jan 23, 2025
8 checks passed
@Yang-33 Yang-33 deleted the Yang-33-patch-1 branch January 23, 2025 02:44
Yang-33 added a commit to line/line-bot-sdk-php that referenced this pull request Jan 23, 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 added a commit to line/line-bot-sdk-python that referenced this pull request Jan 23, 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 added a commit to line/line-bot-sdk-nodejs that referenced this pull request Jan 23, 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 added a commit to line/line-bot-sdk-go that referenced this pull request Jan 23, 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 added a commit to line/line-bot-sdk-ruby that referenced this pull request Jan 23, 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)
# 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.

2 participants