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

Upgrade Framework Script w/ Burn #1084

Merged
merged 173 commits into from
Mar 9, 2025
Merged

Conversation

l-monninger
Copy link
Collaborator

@l-monninger l-monninger commented Mar 5, 2025

Changelog

  • Adds necessary commit hash and script changes to execute the framework upgrade (latest movementxyz/aptos-core) - Calls the destroy_capability_from call (enabled by the upgrade)

Testing

Framework Upgrade is passing and feed overlay transactions are flowing post upgrade

To run the Upgrade locally:

CELESTIA_LOG_LEVEL=FATAL nix develop --extra-experimental-features nix-command --extra-experimental-features flakes --command bash  -c "just movement-full-node native build.setup.eth-local.celestia-local.test-bring-up-elsa-to-biarritz-rc1.feed --keep-tui"

l-monninger and others added 30 commits December 17, 2024 10:38
Provide From conversion impls to construct
PrivateKey and Signature from byte arrays.
The conversions into PrivateKey are fallible, while
a Signature is allowed to be constructed from any
64 bytes.
Add TestSigner providing an in-process Signing
implementation for tests.
TryFromBytes is not needed, neither is the public key
as a member of HashiCorpVault state.
@0xmovses 0xmovses changed the title Upgrade Script w/ Burn Upgrade Framework Script w/ Burn Mar 8, 2025
Copy link
Contributor

@0xmovses 0xmovses left a comment

Choose a reason for hiding this comment

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

🔥 it

@0xmovses
Copy link
Contributor

0xmovses commented Mar 8, 2025

@l-monninger there are two unverified commits here which are yours. Verify you have made these. And then will get @bhenhsi to hit merge (I cannot override the verified commits rule)

@l-monninger l-monninger merged commit 179f2d3 into main Mar 9, 2025
25 of 27 checks passed
mcmillennick pushed a commit that referenced this pull request Mar 9, 2025
Co-authored-by: Andy Golay <andygolay@gmail.com>
Co-authored-by: musitdev <philippe.delrieu@free.fr>
Co-authored-by: Mikhail Zabaluev <mikhail.zabaluev@movementlabs.xyz>
Co-authored-by: Richard Melkonian <35300528+0xmovses@users.noreply.github.com>
Co-authored-by: Icarus131 <anirudhprasad131@protonmail.com>
Co-authored-by: primata <primata@movementlabs.xyz>
Co-authored-by: Icarus131 <anirudhprasad131@gmail.com>
Co-authored-by: Richard Melkonian <r.v.melkonian@gmail.com>
Co-authored-by: Radu Popa <radupopa21be@gmail.com>
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.

6 participants