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: pallet-revive #231

Closed
wants to merge 80 commits into from
Closed

feat: pallet-revive #231

wants to merge 80 commits into from

Conversation

chungquantin
Copy link
Collaborator

@chungquantin chungquantin commented Aug 23, 2024

  • The integration might need to update pop-node to 1.15.0 because pallet-revive is added in the next release. Reference to upgrade pop-node to stable2407, compatible with pallet-revive: chore: upgrade to stable2407 base-parachain#45
  • Until the next version of the pallet-contracts (migrated to PolkaVM), we will just fork the crate and play with it on local.

Daanvdplas and others added 30 commits May 19, 2024 17:38
# This is the 1st commit message:

refactor: general

# This is the commit message #2:

init

# This is the commit message #3:

begin refactor

# This is the commit message #4:

refactor: error handling

# This is the commit message #5:

tests: add error handling tests

# This is the commit message #6:

WIP

# This is the commit message #7:

finalise error handling

# This is the commit message #8:

refactor: easier review
Co-authored-by: Frank Bell <frank@r0gue.io>
@chungquantin chungquantin marked this pull request as draft August 23, 2024 10:15
@peterwht
Copy link
Collaborator

peterwht commented Oct 2, 2024

Just opened #319 with the latest version of pallet-revive. I would say this can be closed now!

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