zerosnacks
zerosnacks
Hi @piohei thanks for your PR! Would you mind updating the PR to resolve the merge conflicts? Looks like there is consensus on the feature and the approach so it...
> Hi @zerosnacks! Sorry for late response. I will try to find some time to resolve conflicts here and prepare it to merge. No problem! Thanks!
Just so I understand correctly, would this supersede https://github.com/foundry-rs/foundry/pull/7135? I quite like the idea of having a keybinding that would trigger the dump to disk from inside of the debugger...
> Just so I understand correctly, would this supersede #7135? > > I quite like the idea of having a keybinding that would trigger the dump to disk from inside...
@piohei would you mind fixing the merge conflicts one last time? I don't have the permissions to push directly to your fork as I normally would.
Merging this, any additional feedback or improvements can be handled in follow-ups Thanks @piohei for your contribution!
Hi @adhusson thanks for flagging For future reference: https://soliditylang.org/blog/2024/10/09/solidity-0.8.28-release-announcement/ this was introduced in 0.8.28 To fix this we'll need an update to Solang: https://crates.io/crates/solang-parser See: https://github.com/hyperledger-solang/solang/issues/1670
Update @adhusson: the Solang team is now aware and adding it should be straightforward, just looking for the time to add it
cc @DaniPopes do we support the `transient` keyword in Solar?
For now marking as `not planned` given that our focus is always on the latest release while given users time to upgrade between major versions. If required you can install...