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

backfix: add signSchnorr to keychain api #173

Merged
merged 2 commits into from
Nov 14, 2024

Conversation

fboucquez
Copy link
Contributor

@fboucquez fboucquez commented Nov 13, 2024

backfix of #171 to be added on top of 7.4.0. Later 7.5.x is not ready for production yet

Not to merge, could we manually publish this?

@mvayngrib
Copy link
Collaborator

we should create a @exodus/keychain@7.4 branch and merge this into that for release, not master

@fboucquez fboucquez changed the base branch from master to @exodus/keychain@7.4 November 14, 2024 08:32
@fboucquez fboucquez merged commit a038120 into @exodus/keychain@7.4 Nov 14, 2024
1 check passed
@fboucquez fboucquez deleted the fboucquez/backfix/keychain-api branch November 14, 2024 08:33
@fboucquez
Copy link
Contributor Author

we should create a @exodus/keychain@7.4 branch and merge this into that for release, not master

Done and merged. I can actually call GH publish action from that branch to release this 7.4.1. It's nice, no local release. I guess we are now run out of excuses for local releases; we could harden this in assets

@mvayngrib
Copy link
Collaborator

we could harden this in assets

please do! You might need @sparten11740 's help to set up release environment protection for those branches

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.

3 participants