Closed sideninja closed 3 months ago
[!CAUTION]
Review failed
The pull request is closed.
The recent changes signify a major refactoring of the codebase, notably shifting from *evmTypes.Block
to *models.Block
, which enhances data handling and readability. Logging mechanisms have been upgraded to use slog
, improving maintainability. These modifications collectively strengthen the functionality, error handling, and overall structure of the system, ensuring it remains robust and user-friendly.
Files | Change Summary |
---|---|
api/api.go , models/events.go , models/transaction.go , storage/index.go , storage/mocks/ReceiptIndexer.go , storage/pebble/receipts.go |
Transitioned from *evmTypes.Block to *models.Block in multiple methods to improve clarity and maintainability in data handling. |
api/server.go , services/logs/filter.go |
Updated logging to utilize slog , enhancing flexibility and maintainability in logging mechanisms. |
go.mod |
Updated dependency versions to keep libraries current, ensuring enhanced functionality and security across the project. |
models/receipt.go |
Introduced a new BloomsHeight struct for better handling of bloom filters and their associated heights. |
tests/web3js/config.js , tests/web3js/eth_deploy_contract_and_interact_test.js , tests/web3js/eth_non_interactive_test.js , tests/web3js/eth_revert_reason_test.js |
Adjusted various test cases and configurations to align with updated blockchain data and structures, ensuring accuracy in assertions. |
storage/index_testsuite.go , services/ingestion/engine_test.go , services/traces/engine_test.go |
Modified tests to reflect changes in block handling and updated structures, improving clarity and functionality in testing frameworks. |
sequenceDiagram
participant Client
participant API
participant Engine
participant Storage
Client->>API: Request Block Data
API->>Engine: Process Request
Engine->>Storage: Retrieve Block
Storage-->>Engine: Return models.Block
Engine-->>API: Provide Block Data
API-->>Client: Return Response
π In a code-filled burrow, where changes abound,
New blocks hop in, with structures profound.
Logging now sings in a streamlined way,
Safety's enhanced for each hop and sway.
With each little tweak, our code leaps with glee,
A brighter, clearer path for all to see! π
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?
Closes: #???
Description
This PR updates flow-go to latest version that includes block mapping and makes corresponding changes to the logic and ensures backward compatibility.
For contributor use:
master
branchFiles changed
in the Github PR explorerSummary by CodeRabbit
New Features
Bug Fixes
Refactor
types.Block
tomodels.Block
across various components for improved data handling and consistency.Tests
Chores