Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rename project "metorite" #19
rename project "metorite" #19
Changes from 2 commits
3436a9f
d843d1c
2ad6830
d7fd227
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
💡 Codebase verification
Found inconsistent package aliases after renaming
The verification revealed that while the import paths have been updated to use "meteorite", there are still package aliases using the old "hardhat" prefix:
broadcast/broadcast.go
:hardhatbank "github.com/somatic-labs/meteorite/modules/bank"
hardhatibc "github.com/somatic-labs/meteorite/modules/ibc"
These aliases are also used in function calls:
hardhatibc.CreateIBCTransferMsg
hardhatbank.CreateBankSendMsg
For consistency, these package aliases should also be updated to use "meteorite" prefix instead of "hardhat".
🔗 Analysis chain
Verify consistent import path updates across the codebase.
The import path update from
hardhat
tometeorite
looks correct. Let's verify this change is consistent across the entire codebase.Let me generate another script to verify the import paths more effectively:
The previous script's output seems incomplete, and we need to verify:
This new script will:
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 339
Script:
Length of output: 1329
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.
💡 Codebase verification
Based on the comprehensive results, I can now provide a final response:
Module rename from 'hardhat' to 'meteorite' is incomplete
The codebase shows an inconsistent state where:
go.mod
still declares "github.com/somatic-labs/hardhat"This inconsistency needs to be addressed by:
go.mod
🔗 Analysis chain
Verify consistent renaming across the codebase.
Let's ensure all imports from the old "hardhat" package have been updated to "meteorite".
The output shows that while there are no direct imports of "hardhat" in Go files, the
go.mod
file still declares the module name as "github.com/somatic-labs/hardhat". Let me verify if this is the only instance and check for any other potential references to ensure we have a complete picture.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 328
Script:
Length of output: 11161