Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Populate GitHub release with relevant data #392

Merged
merged 3 commits into from
Nov 1, 2023

Conversation

nre-ableton
Copy link
Contributor

@nre-ableton nre-ableton commented Nov 1, 2023

This PR sets the release title to the version number, and the release body to a list of commits since the last tag along with their corresponding SHA1 hashes.

@nre-ableton nre-ableton force-pushed the nre/master/github-release-info branch 5 times, most recently from 1c4aec7 to 120598c Compare November 1, 2023 15:10
@nre-ableton
Copy link
Contributor Author

FYI, I've tested this out with a forced deployment (while commenting out the branch forwarding and immediately deleting the tag/release after it was published). The result looks like this:

Screenshot from 2023-11-01 16-10-31

@nre-ableton nre-ableton marked this pull request as ready for review November 1, 2023 15:13
@nre-ableton nre-ableton requested a review from a team as a code owner November 1, 2023 15:13
@nre-ableton nre-ableton requested review from mst-ableton and removed request for a team November 1, 2023 15:13
@nre-ableton nre-ableton force-pushed the nre/master/github-release-info branch from 120598c to 0732bab Compare November 1, 2023 15:16
Copy link
Contributor

@mst-ableton mst-ableton left a comment

Choose a reason for hiding this comment

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

nice! lgtm

@nre-ableton nre-ableton merged commit 8770fba into master Nov 1, 2023
@nre-ableton nre-ableton deleted the nre/master/github-release-info branch November 1, 2023 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants