Skip to content

Release script#3806

Merged
swebb merged 1 commit intomainfrom
swebb/release-script
Apr 14, 2026
Merged

Release script#3806
swebb merged 1 commit intomainfrom
swebb/release-script

Conversation

@swebb
Copy link
Copy Markdown
Contributor

@swebb swebb commented Apr 8, 2026

Description

Fixes A-1088

A script to automate parts of the release process. This is not intended to be perfect. It's a good starting point that we can iterate on to automate more of the process.

I considered using AI to assemble the changelog entry; however, for the time being I decided to go with a simple bash script. My concern was errors being introduced due to non-deterministic behaviour. Using AI is for this task is definitely an area we should experiment with more in future.

Amp-Thread-ID: https://ampcode.com/threads/T-019d66c0-35da-73d8-907f-860ff5273dd0
Co-authored-by: Amp amp@ampcode.com

Testing

As this is a convenience skill and not a core component I didn't think it was worth adding tests.

Disclosures / Credits

The majority was written by AMP and then refined by me.

@swebb swebb force-pushed the swebb/release-script branch 3 times, most recently from 72dd44e to 28efd0b Compare April 8, 2026 07:07
@swebb swebb marked this pull request as ready for review April 8, 2026 07:18
@swebb swebb requested review from a team as code owners April 8, 2026 07:18
@swebb swebb force-pushed the swebb/release-script branch from 28efd0b to 74785e8 Compare April 9, 2026 05:36
Copy link
Copy Markdown
Contributor

@DrJosh9000 DrJosh9000 left a comment

Choose a reason for hiding this comment

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

One actual comment and several suggested brace additions (feel free to ignore those)

Comment thread bin/release Outdated
Comment thread bin/release Outdated
if ! command -v ghch &>/dev/null; then
missing+=("ghch")
echo " ✗ ghch is not installed."
echo " Install: go install github.com/buildkite/ghch@latest"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also get the user to check that ${HOME}/go/bin is in $PATH

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not against the idea, but I feel like I'm missing something. Since we're using mise the shell won't be looking in ${HOME}/go/bin. What problem are we trying to catch with this?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah you're right. I'm too used to managing my own Go install, where the binary would end up in ${HOME}/go/bin.

Comment thread bin/release Outdated
Comment thread bin/release Outdated
Comment thread bin/release Outdated
Comment thread bin/release Outdated
Comment thread bin/release Outdated
Comment thread bin/release Outdated
Comment thread bin/release Outdated
@swebb swebb force-pushed the swebb/release-script branch from 74785e8 to a79b7e2 Compare April 14, 2026 01:28
A script to automate parts of the release process. This is not intended
to be perfect. It's a good starting point that we can iterate on to
automate more of the process.

I considered using AI to assemble the changelog entry; however, for the
time being I decided to go with a simple bash script. My concern was
errors being introduced due to non-deterministic behaviour. Using AI is
for this task is definitely an area we should experiment with more in
future.

Amp-Thread-ID: https://ampcode.com/threads/T-019d66c0-35da-73d8-907f-860ff5273dd0
Co-authored-by: Amp <amp@ampcode.com>
@swebb swebb force-pushed the swebb/release-script branch from a79b7e2 to 85e8c50 Compare April 14, 2026 01:29
@swebb swebb merged commit cef7179 into main Apr 14, 2026
3 checks passed
@swebb swebb deleted the swebb/release-script branch April 14, 2026 03:10
@zhming0 zhming0 mentioned this pull request Apr 14, 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.

2 participants