Skip to content

feat: share release notes generation between release workflows#431

Merged
doron050 merged 1 commit into
mainfrom
shared-release-notes
Jul 25, 2026
Merged

feat: share release notes generation between release workflows#431
doron050 merged 1 commit into
mainfrom
shared-release-notes

Conversation

@doron050

Copy link
Copy Markdown
Collaborator

Summary

Extracts release notes generation into a shared script scripts/generate-release-notes.sh used by both gen-release-pr.yml and release.yml, ensuring consistent output.

Changes

scripts/generate-release-notes.sh (new)

  • Shared script that generates formatted release notes from PRs between tags
  • Filters PRs by label: only includes major/minor/patch labeled PRs, excludes skip-release
  • Lists actual contributors dynamically (no more hardcoded @SockworkOrange)

.github/workflows/gen-release-pr.yml

  • Release notes preview now uses the shared script → preview matches actual release exactly
  • PR list entries show just #XYZ (GitHub auto-links) instead of full title
  • Preview wrapped in a collapsible <details> section
  • Changelog in preview only shows labeled PRs (matching what release.yml will publish)

.github/workflows/release.yml

  • Replaced the inline release notes construction with a call to the shared script
  • Contributors are now dynamic (pulled from PR authors instead of hardcoded)

…se workflows

- Create scripts/generate-release-notes.sh that filters PRs by label
  (major/minor/patch, excluding skip-release) and outputs formatted notes
- Update gen-release-pr.yml to use the shared script for the collapsible
  release notes preview in the PR body
- Update release.yml to use the shared script for actual release notes
- Contributors list is now dynamic (no longer hardcoded @SockworkOrange)
@github-actions

Copy link
Copy Markdown
Contributor

Release Assistant - Passed

No version-worthy changes detected. Only non-functional, doc, or test changes.

@doron050
doron050 merged commit 8bea7f6 into main Jul 25, 2026
7 checks passed
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.

1 participant