Skip to content

Conversation

@dscho
Copy link
Member

@dscho dscho commented Nov 4, 2025

There is a new initiative to increase confidence in GitHub Releases. The official guidance is to create a draft release, then attach all assets, then publish.

We almost do that already. The only exception is the additional update to create a discussion for each new release. This was separated out earlier in response to a missing permission of the GitHub App used to perform the job, but is likely incompatible with immutable releases. So let's revert b59f512 (github-release: create the Discussion separately, 2024-09-18).

There is a new initiative to increase confidence in GitHub Releases:
https://github.blog/changelog/2025-10-28-immutable-releases-are-now-generally-available/

The official guidance is to create a draft release, then attach all assets, then publish:
https://docs.github.com/en/code-security/supply-chain-security/understanding-your-software-supply-chain/immutable-releases#best-practices-for-publishing-immutable-releases

We _almost_ do that already. The only exception is the additional update
to create a discussion for each new release. This was separated out
earlier in response to a missing permission of the GitHub App used to
perform the job, but is likely incompatible with immutable releases.

So let's revert b59f512 (github-release: create the Discussion
separately, 2024-09-18).

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho requested review from mjcheetham and rimrul November 4, 2025 11:53
@dscho dscho self-assigned this Nov 4, 2025
@dscho dscho merged commit 56d8604 into main Nov 4, 2025
@dscho dscho deleted the prepare-for-immutable-releases branch November 4, 2025 12:57
@dscho
Copy link
Member Author

dscho commented Nov 5, 2025

I guess I should have done this before releasing Git for Windows v2.52.0-rc0 ;-)

image

At least -rc1 will now exercise this.

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