grandizzy

Results 349 comments of grandizzy

this happens because storage hashes are not dumped / reloaded from state file https://github.com/foundry-rs/foundry/blob/9148dbc2fe0c72e249669b11d04caae593fa6113/crates/anvil/src/eth/backend/mem/mod.rs#L1474 @mattsse would be ok to include them? le: it will also require persisting blocks which cannot...

can take this one, using `serde_json::Map` (which is backed by a BTreeMap) here https://github.com/foundry-rs/foundry/blob/master/crates/cheatcodes/src/json.rs#L371 should solve it

> can take this one, using `serde_json::Map` (which is backed by a BTreeMap) here https://github.com/foundry-rs/foundry/blob/master/crates/cheatcodes/src/json.rs#L371 should solve it hm, actually this will probably break backward compatibility so don't think it's...

> I believe the limitation @lightclient ran into was that wallet ls only targets the local keystores. > > maybe we add an `--all` option to also check connected wallets,...

> This makes sense to me. made a draft PR https://github.com/foundry-rs/foundry/pull/7123 looking for comments

fixed in #7123 @mattsse this can be closed

> > Does this change resolve the issue or further dev work is still needed? I had a look at this and it's still reproducible but if I am not...

> yeah I think the best solution is to always assume the fork needs to be created if the block_number is none or latest and then we check afterwards if...

> @grandizzy feel free to take this :) > > ty! thanks, I made a draft PR with a proposed solution https://github.com/foundry-rs/foundry/pull/7087 please have a look when you have time

https://github.com/foundry-rs/foundry/pull/7595 fix is available, @mattsse this one can be closed