-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Update to new fuel-vm version #917
Conversation
Hmm, it looks like it almost duplicates the #806. Maybe it is better to use Tommi's PR as a base for your PR to include FuelLabs/fuel-vm#307 |
I totally missed that PR, it indeed seems to have lots of duplicate changes. I'm going to wait until it's merged and rebase this on top of it after. |
Adapted implementation of the `StorageInspect` and `StorageMutate` to use a new `Mappable`. Used a new double storage key instead of tuples. `empty_sum` is private now=( Repalced with `empty_sum_sha256`. Will create a patch for `fuel-merkle`.
14401dd
to
41fb59e
Compare
|
||
#[test] | ||
fn get() { | ||
let balance_id: (ContractId, AssetId) = | ||
(ContractId::from([1u8; 32]), AssetId::new([1u8; 32])); | ||
let key = (&ContractId::from([1u8; 32]), &AssetId::new([1u8; 32])).into(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Includes breaking changes from FuelLabs/fuel-vm#307. Requires FuelLabs/fuel-vm#314 as well.