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

refresh-manifests workflow checks all platforms before committing manifests #73

Merged
merged 7 commits into from
Jun 2, 2023

Conversation

mitchmindtree
Copy link
Contributor

Closes #70
Closes #71

The new workflow has three main jobs:

  1. Refresh the manifests using the refresh-manifests.sh script and upload the manifests directory to temporary workflow storage.
  2. If 1. succeeds: For all supported platforms and all major package sets (fuel, fuel-nightly, fuel-beta-3, etc), download the new manifests from temporary storage, add them to git staging so they get picked up by nix, then do a test nix build. Cache the binaries on success.
  3. If 2. succeeds: Download the manifests from temporary storage, commit them to master and push to the main master branch.

mitchmindtree added a commit that referenced this pull request Jun 2, 2023
This reverts commit 688f10a, a commit
accidentally pushed from CI while testing #73.
@mitchmindtree mitchmindtree force-pushed the mitchmindtree/fix-refresh-manifests-checks branch from 93c89e3 to d3d00c2 Compare June 2, 2023 05:29
@mitchmindtree mitchmindtree marked this pull request as ready for review June 2, 2023 06:10
@mitchmindtree
Copy link
Contributor Author

OK, after a few tweaks to the workflow (and an accidental master commit 😅) this should now be ready to go 👍

@mitchmindtree mitchmindtree merged commit aa7f14c into master Jun 2, 2023
@mitchmindtree mitchmindtree deleted the mitchmindtree/fix-refresh-manifests-checks branch June 2, 2023 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants