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

feat(ci): Add builds for preview binaries #1777

Merged
merged 5 commits into from
Mar 22, 2023
Merged

Conversation

phated
Copy link
Member

@phated phated commented Mar 22, 2023

This updates the Release workflow to generate "preview" binaries whenever the Release Please updates the release PR. It also uses that binary to regenerate and push the docs on the release branch, which triggers a CI run necessary for us to merge the release in the queue.

I refactored the workflow a bit so we use the same logic to build and upload the binaries with the same workflow. This should allow us to be more confident the release flow will happen successfully since the previews are built often.

I didn't get around to adding workflow_dispatch so we could run these in isolation, which is mostly because we rely on the binary being an artifact of a run. We might need to combine some reusable workflows into a dispatch, but I'm not sure yet.

@phated phated requested a review from ospencer as a code owner March 22, 2023 08:58
@phated phated self-assigned this Mar 22, 2023
@phated phated added this pull request to the merge queue Mar 22, 2023
@phated phated merged commit d512c5a into main Mar 22, 2023
@phated phated deleted the phated/preview-binaries branch March 22, 2023 16:29
av8ta pushed a commit to av8ta/grain that referenced this pull request Apr 11, 2023
* feat(ci): Add builds for preview binaries

* chore(ci): Update actions

* chore(ci): Only run the CI workflow on PR, merge_group, and push-to-main

* document decision

* Remove seemingly unnecessary string quotes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants