-
Notifications
You must be signed in to change notification settings - Fork 16
chore: rename residual QGB mentions to Blobstream #622
chore: rename residual QGB mentions to Blobstream #622
Conversation
WalkthroughThe changes across various Go files reflect a rebranding or refactoring effort, where references to "QGB" are being replaced with "Blobstream". This includes updates to log messages, directory names, import paths, and chain IDs. The modifications are textual and do not alter the core functionality or structure of the codebase. Changes
TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (6)
- benchmark/main.go (2 hunks)
- evm/evm_client.go (2 hunks)
- orchestrator/orchestrator_test.go (3 hunks)
- relayer/relayer_test.go (4 hunks)
- store/init.go (2 hunks)
- testing/celestia_network.go (1 hunks)
Files skipped from review due to trivial changes (5)
- benchmark/main.go
- evm/evm_client.go
- orchestrator/orchestrator_test.go
- relayer/relayer_test.go
- store/init.go
Additional comments: 2
testing/celestia_network.go (2)
91-94: The change in chain ID from "qgb-test" to "blobstream-test" is consistent with the pull request's objective to rename QGB to Blobstream. Ensure that all references and dependencies throughout the codebase that rely on the chain ID are updated accordingly.
91-97: > Note: This review was outside the patches, and no patch overlapping with it was found. Original lines [7-7]
The import path
github.com/celestiaorg/celestia-app/x/qgb/types
should be updated to reflect the new naming convention if the module has been renamed accordingly. Verify that the module path is correct and update if necessary.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #622 +/- ##
=======================================
Coverage 26.13% 26.13%
=======================================
Files 29 29
Lines 3019 3019
=======================================
Hits 789 789
Misses 2135 2135
Partials 95 95 ☔ View full report in Codecov by Sentry. |
Overview
Checklist
Summary by CodeRabbit
Refactor
Tests
blobstreamtesting
package.Chores