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

Move release process to be based off of tagging instead of pushes to main #2141

Merged
merged 10 commits into from
Jun 14, 2024

Conversation

plorenz
Copy link
Member

@plorenz plorenz commented Jun 13, 2024

No description provided.

@plorenz plorenz requested review from a team as code owners June 13, 2024 13:24
name: release

on:
push:
Copy link
Member

Choose a reason for hiding this comment

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

not release trigger?

Copy link
Member Author

Choose a reason for hiding this comment

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

Tag creates the release, marked as pre-release. Could do it the other way, but it's even more manual

Copy link
Member

Choose a reason for hiding this comment

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

don't we want release to be a deliberate manual process?
either way is fine

Copy link
Member Author

Choose a reason for hiding this comment

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

It is a deliberate manual process, it's just one done via tag, push tag, rather than through the GH UI.

RELEASING.md Outdated
(`isLatest: true`).

1. After an arbitrary burn-in period, unmark "prerelease" in GitHub Releases (`isPrerelease: false`) to stop
discouraging users from consuming the now-stable release. This will automatically promote and advertise the downstreams.
Copy link
Member

Choose a reason for hiding this comment

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

is this supposed to be non-stable?

stop discouraging reads like double negative, I had to think what it means

Copy link
Member

Choose a reason for hiding this comment

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

Let's drop the confusing part. The intent is to emphasize that setting isPrerelease: false triggers promoting downstreams, not isLatest: true.

to stop discouraging users from consuming the now-stable release

@plorenz plorenz merged commit 75b360d into release-next Jun 14, 2024
11 of 12 checks passed
@plorenz plorenz deleted the tag-based-releases branch June 14, 2024 18:39
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.

None yet

3 participants