Skip to content

fix(release): use correct template variable in pr_body#178

Merged
zeitlinger merged 1 commit intomainfrom
fix/release-plz-pr-body
Apr 16, 2026
Merged

fix(release): use correct template variable in pr_body#178
zeitlinger merged 1 commit intomainfrom
fix/release-plz-pr-body

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

Summary

  • {{ changelog }} is only valid in git_release_body, not pr_body
  • release-plz bails with Variable 'changelog' not found in context while rendering 'pr_body' (run)
  • Fix: use release.changelog inside a releases loop

Test plan

  • Merge and verify release-plz "Release PR" job succeeds on next push to main

`{{ changelog }}` is only valid in `git_release_body`.
For `pr_body` the correct variable is `release.changelog` inside a
releases loop.
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger zeitlinger marked this pull request as ready for review April 16, 2026 14:51
@zeitlinger zeitlinger requested a review from a team as a code owner April 16, 2026 14:51
Copilot AI review requested due to automatic review settings April 16, 2026 14:51
@zeitlinger zeitlinger enabled auto-merge (squash) April 16, 2026 14:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the release-plz PR body template to use the correct template context so the release PR job no longer fails during rendering.

Changes:

  • Replace invalid {{ changelog }} usage in pr_body with a loop over releases and {{ release.changelog }}.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zeitlinger zeitlinger merged commit d246508 into main Apr 16, 2026
18 checks passed
@zeitlinger zeitlinger deleted the fix/release-plz-pr-body branch April 16, 2026 15:00
@github-actions github-actions bot mentioned this pull request Apr 16, 2026
zeitlinger added a commit that referenced this pull request Apr 16, 2026
### Added

- *(release)* migrate from release-please to release-plz
([#171](#171))

### Fixed

- *(release)* use correct template variable in pr_body
([#178](#178))
- *(release)* suppress component prefix in release-please tags
([#166](#166))
- *(release)* add workflow_dispatch to retrigger for existing tags
([#167](#167))

### Other

- move icon to assets/ to fix release-plz
([#177](#177))
- *(deps)* update dependency npm:renovate to v43.102.11 [security]
([#174](#174))
- *(deps)* update rust crate similar to v3.1.0
([#173](#173))
- *(deps)* update dependency github:mvdan/sh to v3.13.1
([#163](#163))

> [!IMPORTANT]
> Close and reopen this PR to trigger CI checks.

---------

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
This was referenced Apr 16, 2026
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.

3 participants