Skip to content

ci(release): preview notes on pull requests and list every change - #302

Merged
arrowplum merged 7 commits into
mainfrom
ci/release-notes-preview-and-completeness
Aug 22, 2026
Merged

ci(release): preview notes on pull requests and list every change#302
arrowplum merged 7 commits into
mainfrom
ci/release-notes-preview-and-completeness

Conversation

@arrowplum

@arrowplum arrowplum commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Release notes are only visible after publishing, they omit part of what shipped, and the release goes out with no chance to review it.

Follows INFRA-678.

Changes

  • release.yaml already triggers on pull_request but prints nothing useful. It now runs a dry run and writes the projected version and notes to the job summary.
  • presetConfig.types unhides the types the conventionalcommits preset drops, so dependency and CI commits are listed. Listing only; the version bump is still commit-analyzer.releaseRules.
  • draftRelease: true, so the release is created as a draft for a human to publish.

Known gap

A non-conventional commit subject is not categorised and so is not listed. #292 is the only one in v4.1.0, and StepSecurity pull requests bypass title validation in reusable_pr-hygiene.yml, so it recurs on theirs. Attribution is out of reach for the same reason: a commit carries a name and email, not a GitHub handle.

Test plan

  • Dry run gives 4.1.0, unchanged, with a new Build System section
  • Preview renders on this PR, in the Release job summary

Signed-off-by: Joe Martin <jmartin@aerospike.com>
@arrowplum
arrowplum requested a review from a team as a code owner August 13, 2026 00:01
Signed-off-by: Joe Martin <jmartin@aerospike.com>
@arrowplum
arrowplum marked this pull request as draft August 13, 2026 00:06
Signed-off-by: Joe Martin <jmartin@aerospike.com>
Signed-off-by: Joe Martin <jmartin@aerospike.com>
Signed-off-by: Joe Martin <jmartin@aerospike.com>
@arrowplum
arrowplum marked this pull request as ready for review August 13, 2026 00:45
Signed-off-by: Joe Martin <jmartin@aerospike.com>
@arrowplum
arrowplum force-pushed the ci/release-notes-preview-and-completeness branch from d83ec7c to f2cc74c Compare August 13, 2026 01:29
albertlagman
albertlagman previously approved these changes Aug 13, 2026
Signed-off-by: Joe Martin <jmartin@aerospike.com>

@mcbridem mcbridem left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Neat.

@arrowplum
arrowplum merged commit ee3240d into main Aug 22, 2026
57 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.

3 participants