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

[wallet-ext] New wallet versioning strategy #5814

Merged
merged 4 commits into from
Nov 9, 2022

Conversation

Jordan-Mysten
Copy link
Contributor

@Jordan-Mysten Jordan-Mysten commented Nov 3, 2022

This updates the wallet versioning strategy so that it is aligned with the new wallet release proposal. This includes support for beta builds, and better name labelling for the dev builds.

I removed the wallet version workflow as we're now accomplishing this through a different means (date-based versioning + beta builds).

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2022

💳 Wallet Extension has been built, you can download the packaged extension here: https://github.com/MystenLabs/sui/actions/runs/3424442057#artifacts

@Jordan-Mysten Jordan-Mysten changed the title [wip][wallet-ext] Wallet beta builds [wip][wallet-ext] New wallet versioning strategy Nov 7, 2022
@Jordan-Mysten Jordan-Mysten changed the title [wip][wallet-ext] New wallet versioning strategy [wallet-ext] New wallet versioning strategy Nov 7, 2022
@Jordan-Mysten Jordan-Mysten force-pushed the jordan--wallet-beta-builds branch from 529d317 to 70270e1 Compare November 7, 2022 20:48
@Jordan-Mysten Jordan-Mysten marked this pull request as ready for review November 7, 2022 20:48
}

const WALLET_BETA = !!process.env.WALLET_BETA;
const PATCH_VERISON = Number(process.env.PATCH_VERSION) || 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this used in case we need to publish again the same day?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah exactly, just buys us the ability to do multiple releases in a day in the event something goes wrong. Or if we do want a true patch release of a dated version, we can just cherry-pick commits onto it and issue a patch vs having to issue a full version.

@vercel
Copy link

vercel bot commented Nov 9, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
explorer ✅ Ready (Inspect) Visit Preview Nov 9, 2022 at 0:52AM (UTC)

@Jordan-Mysten Jordan-Mysten merged commit 657caa2 into main Nov 9, 2022
@Jordan-Mysten Jordan-Mysten deleted the jordan--wallet-beta-builds branch November 9, 2022 00:57
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.

4 participants