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: deploy using version numbers for binaries #12

Merged
merged 1 commit into from Sep 27, 2023

Conversation

jacderida
Copy link
Contributor

Optionally supply version numbers, which will deploy the binaries built during the release process. We still need to build the faucet binary because that's not a released artifact at the moment.

When the version number arguments are used, we skip building the safenode and safe binaries, which does shave a decent amount of time off the overall process.

The other significant difference is that we pull the version binaries from the Github release rather than S3, so we had to introduce another mechanism for downloading those. At the moment the release process does not include uploading the versioned binaries to S3. Only the 'latest' version goes up there.

Optionally supply version numbers, which will deploy the binaries built during the release process.
We still need to build the faucet binary because that's not a released artifact at the moment.

When the version number arguments are used, we skip building the `safenode` and `safe` binaries,
which does shave a decent amount of time off the overall process.

The other significant difference is that we pull the version binaries from the Github release rather
than S3, so we had to introduce another mechanism for downloading those. At the moment the release
process does not include uploading the versioned binaries to S3. Only the 'latest' version goes up
there.
@joshuef joshuef merged commit 22cf98e into maidsafe:main Sep 27, 2023
7 checks passed
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

2 participants