Skip to content

Adjust reading release message from tag for new $GITHUB_OUTPUT#1186

Merged
simu merged 1 commit into
masterfrom
fix/release-message-from-tag
Jul 29, 2025
Merged

Adjust reading release message from tag for new $GITHUB_OUTPUT#1186
simu merged 1 commit into
masterfrom
fix/release-message-from-tag

Conversation

@simu

@simu simu commented Jul 29, 2025

Copy link
Copy Markdown
Member

Passing multi-line strings between action steps has changed with the new $GITHUB_OUTPUT variable compared to ::set-output.

We now don't escape newlines, but instead encode the whole string as JSON and decode it with GitHub actions' builtin fromJSON() to pass it to ncipollo/release-action.

Additionally, we've updated the sed expression to preserve list items as long as the '*' is right at the start of the new line.

Tested in

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency, internal
    as they show up in the changelog

Passing multi-line strings between action steps has changed with the new
`$GITHUB_OUTPUT` variable compared to `::set-output`.

We now don't escape newlines, but instead encode the whole string as
JSON and decode it with GitHub actions' builtin `fromJSON()` to pass it
to `ncipollo/release-action`.
@simu simu requested a review from a team as a code owner July 29, 2025 07:45
@simu simu added the ignore Ignore PR in changelog label Jul 29, 2025
@simu simu requested a review from a team July 29, 2025 07:52
@simu simu merged commit 7da60a6 into master Jul 29, 2025
20 checks passed
@simu simu deleted the fix/release-message-from-tag branch July 29, 2025 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore Ignore PR in changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants