Tristav

Results 8 comments of Tristav

@damian-orzechowski - Removed hard validation for boundary limits (`OutOfBounds`) — accounts and slots outside the range are now handled gracefully using `hasExtraStorage` logic. - Kept sorting validation for `accounts` and...

@damian-orzechowski Added unit tests for the sorting validation as requested

@damian-orzechowski Just a quick follow-up — I've added the requested unit tests in the latest commit. Please let me know if anything else needs to be adjusted. Thanks for your...

@LukaszRozmej Hi! Just noting that the PR looks good otherwise, but two CI checks are still failing: - `Nethermind.Synchronization.Test` - `Nethermind tests`

Hi @asdacap, @damian-orzechowski, @rubo Just a friendly reminder about PR #8551: it adds sorting order validation and boundary checks for accounts and storage slots in `SnapProviderHelper`, introduces `AddRangeResult::InvalidOrder` and `OutOfBounds`,...

I've encountered test execution issues during local development - tests hang indefinitely in the Synchronization.Test suite. To ensure PR stability and avoid CI failures, I've temporarily removed the unit tests...

Hi @solidity-dev team! Just a friendly reminder about PR #15913: it fixes an `AttributeError` in `insert_node_before` by replacing: child.parent.insert(position, new_sibling) with child.parent.children.insert(position, new_sibling) This ensures proper insertion of nodes under...

Hi @AdityaSripal, @angbrav Just a friendly reminder about PR #1209: it fixes TypeScript syntax in the upgrade callback functions in `UPGRADES.md` by: - Changing `[Identifier]` to `Identifier[]` - Updating arrow...