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

Publish GitHub Releases #136

Merged
merged 1 commit into from
Jun 21, 2022
Merged

Publish GitHub Releases #136

merged 1 commit into from
Jun 21, 2022

Conversation

mmlb
Copy link
Contributor

@mmlb mmlb commented Jun 21, 2022

Description

Publishes GitHub Releases for both version tags and pushes to main.

Why is this needed

Publishing on main to the latest release gives us a url we can use for sandbox
until a new release is cut if we want to run sandbox with bleeding edge
hook builds.

Publishing on version tags (v*) removes some friction to creating releases and hopefully opens us up for more frequent releases.

How Has This Been Tested?

I simulated many of the scenarios on my fork, see the action runs for it https://github.com/mmlb/tinkerbell-hook/actions.

How are existing users impacted? What migration steps/scripts do we need?

Usable hooks w/o needing to build locally + hopefully more frequent tagged releases.

On both push to main (updates the `latest` release) and tags (creates release
for the tag).

Publishing on main to the latest release gives us a url we can use for sandbox
until a new release is cut or if we want to run sandbox with bleeding edge
hook builds.

Signed-off-by: Manuel Mendez <[email protected]>
@mmlb mmlb requested review from ScottGarman and detiber June 21, 2022 14:37
@mmlb mmlb removed the request for review from detiber June 21, 2022 18:11
@mmlb mmlb added the ready-to-merge Signal to Mergify to merge the PR. label Jun 21, 2022
@mergify mergify bot merged commit 494f99a into tinkerbell:main Jun 21, 2022
@mmlb mmlb deleted the github-releases branch June 21, 2022 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants