Closed sideninja closed 3 weeks ago
[!CAUTION]
Review failed
The pull request is closed.
The changes involve the removal of legacy block event decoding logic from the decodeBlockEvent
function in the models/block.go
file. The fallback mechanism for legacy decoding has been eliminated, simplifying error handling. Additionally, the decodeLegacyBlockEvent
function and its associated type, blockEventPayloadV0
, have been removed, indicating a transition away from legacy support.
Files | Change Summary |
---|---|
models/block.go | Removed decodeLegacyBlockEvent function and blockEventPayloadV0 type; eliminated legacy decoding logic from decodeBlockEvent . |
In the garden of code, I hop with glee,
The old ways are gone, oh what a spree!
No more tangled paths, just clear skies ahead,
With each leap I take, I'm light as a thread.
Hooray for the changes, let’s dance and play,
In a world that's modern, hip-hip-hooray! 🐇✨
[!TIP] We have updated our review workflow to use the Anthropic's Claude family of models. Please share any feedback in the discussion post on our Discord.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Reverts onflow/flow-evm-gateway#475
Merge once this is merged https://github.com/onflow/flow-go/pull/6398
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
decodeBlockEvent
function for better maintainability and clarity.