-
Notifications
You must be signed in to change notification settings - Fork 0
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
Troubleshoot GH action #1
base: main
Are you sure you want to change the base?
Conversation
WalkthroughThis update reflects a comprehensive rebranding effort, migrating import paths and module names from the old namespace Changes
Poem
Tip Early access features: enabledWe are currently testing the following features in early access:
Note:
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? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Actionable comments posted: 0
Outside diff range and nitpick comments (2)
proto/generate.sh (2)
Line range hint
1-1
: Add a shebang line.Shell scripts should start with a shebang line to specify the interpreter.
+#!/bin/bash
Line range hint
1-1
: Add error handling for thecd
command.Use
cd ... || exit
orcd ... || return
in case thecd
command fails.-cd proto +cd proto || exit
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (8)
x/aura/types/blocklist/events.pb.go
is excluded by!**/*.pb.go
x/aura/types/blocklist/genesis.pb.go
is excluded by!**/*.pb.go
x/aura/types/blocklist/query.pb.go
is excluded by!**/*.pb.go
x/aura/types/blocklist/tx.pb.go
is excluded by!**/*.pb.go
x/aura/types/events.pb.go
is excluded by!**/*.pb.go
x/aura/types/genesis.pb.go
is excluded by!**/*.pb.go
x/aura/types/query.pb.go
is excluded by!**/*.pb.go
x/aura/types/tx.pb.go
is excluded by!**/*.pb.go
Files selected for processing (40)
- .github/workflows/e2e-tests.yaml (1 hunks)
- e2e/go.mod (2 hunks)
- e2e/utils.go (1 hunks)
- go.mod (1 hunks)
- proto/aura/blocklist/v1/events.proto (1 hunks)
- proto/aura/blocklist/v1/genesis.proto (1 hunks)
- proto/aura/blocklist/v1/query.proto (1 hunks)
- proto/aura/blocklist/v1/tx.proto (1 hunks)
- proto/aura/v1/events.proto (1 hunks)
- proto/aura/v1/genesis.proto (1 hunks)
- proto/aura/v1/query.proto (1 hunks)
- proto/aura/v1/tx.proto (1 hunks)
- proto/generate.sh (1 hunks)
- simapp/app.go (1 hunks)
- simapp/go.mod (2 hunks)
- simapp/simd/cmd/root.go (1 hunks)
- simapp/simd/main.go (1 hunks)
- utils/mocks/aura.go (1 hunks)
- utils/mocks/bank.go (1 hunks)
- x/aura/client/cli/query.go (1 hunks)
- x/aura/client/cli/query_blocklist.go (1 hunks)
- x/aura/client/cli/tx.go (1 hunks)
- x/aura/client/cli/tx_blocklist.go (1 hunks)
- x/aura/genesis.go (1 hunks)
- x/aura/keeper/keeper.go (1 hunks)
- x/aura/keeper/keeper_test.go (1 hunks)
- x/aura/keeper/msg_server.go (1 hunks)
- x/aura/keeper/msg_server_blocklist.go (1 hunks)
- x/aura/keeper/msg_server_blocklist_test.go (1 hunks)
- x/aura/keeper/msg_server_test.go (1 hunks)
- x/aura/keeper/query_server.go (1 hunks)
- x/aura/keeper/query_server_blocklist.go (1 hunks)
- x/aura/keeper/query_server_blocklist_test.go (1 hunks)
- x/aura/keeper/query_server_test.go (1 hunks)
- x/aura/keeper/state.go (1 hunks)
- x/aura/keeper/state_blocklist.go (1 hunks)
- x/aura/keeper/state_blocklist_test.go (1 hunks)
- x/aura/module.go (1 hunks)
- x/aura/types/codec.go (1 hunks)
- x/aura/types/genesis.go (1 hunks)
Files skipped from review due to trivial changes (14)
- go.mod
- proto/aura/blocklist/v1/events.proto
- proto/aura/v1/events.proto
- proto/aura/v1/genesis.proto
- x/aura/client/cli/query.go
- x/aura/client/cli/query_blocklist.go
- x/aura/client/cli/tx.go
- x/aura/keeper/keeper.go
- x/aura/keeper/msg_server_test.go
- x/aura/keeper/query_server_blocklist_test.go
- x/aura/keeper/query_server_test.go
- x/aura/keeper/state.go
- x/aura/keeper/state_blocklist.go
- x/aura/types/genesis.go
Additional context used
Shellcheck
proto/generate.sh
[error] 1-1: Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.
(SC2148)
[warning] 1-1: Use 'cd ... || exit' or 'cd ... || return' in case cd fails.
(SC2164)
Additional comments not posted (30)
proto/generate.sh (1)
5-5
: LGTM!The updated copy command reflects the new module path and looks correct.
proto/aura/blocklist/v1/genesis.proto (1)
5-5
: LGTM!The updated
go_package
option reflects the new module path and looks correct.x/aura/keeper/state_blocklist_test.go (1)
6-7
: LGTM!The updated import paths reflect the new module path and look correct.
utils/mocks/aura.go (1)
11-12
: LGTM!The updated import paths reflect the new module path and look correct.
proto/aura/blocklist/v1/query.proto (1)
8-8
: Update import paths consistently.The
go_package
option has been updated to the new module path. Ensure all related paths and references are also updated consistently throughout the codebase.x/aura/genesis.go (1)
5-7
: Update import paths consistently.The import paths have been updated to the new module path. Ensure all related paths and references are also updated consistently throughout the codebase.
simapp/simd/main.go (1)
9-10
: Update import paths consistently.The import paths have been updated to the new module path. Ensure all related paths and references are also updated consistently throughout the codebase.
proto/aura/v1/query.proto (1)
9-9
: Update import paths consistently.The
go_package
option has been updated to the new module path. Ensure all related paths and references are also updated consistently throughout the codebase.proto/aura/blocklist/v1/tx.proto (1)
7-7
: Update go_package path.The
go_package
option has been updated to the new module pathgithub.com/ondoprotocol/usdy-noble/x/aura/types/blocklist
.x/aura/keeper/query_server_blocklist.go (1)
11-11
: Update import path.The import path has been updated to the new module path
github.com/ondoprotocol/usdy-noble/x/aura/types/blocklist
.x/aura/keeper/query_server.go (1)
8-8
: Update import path.The import path has been updated to the new module path
github.com/ondoprotocol/usdy-noble/x/aura/types
..github/workflows/e2e-tests.yaml (1)
19-19
: Update GitHub Action path and version.The
uses
directive has been updated to usestrangelove-ventures/[email protected]
.utils/mocks/bank.go (1)
8-8
: Import Path Update: Correctness and ConsistencyThe import path has been updated to reflect the new module structure. Ensure that all references to
github.com/noble-assets/aura/x/aura/types
are correctly replaced withgithub.com/ondoprotocol/usdy-noble/x/aura/types
across the codebase.x/aura/types/codec.go (1)
8-8
: Import Path Update: Correctness and ConsistencyThe import path has been updated to reflect the new module structure. Ensure that all references to
github.com/noble-assets/aura/x/aura/types/blocklist
are correctly replaced withgithub.com/ondoprotocol/usdy-noble/x/aura/types/blocklist
across the codebase.x/aura/client/cli/tx_blocklist.go (1)
7-7
: Import Path Update: Correctness and ConsistencyThe import path has been updated to reflect the new module structure. Ensure that all references to
github.com/noble-assets/aura/x/aura/types/blocklist
are correctly replaced withgithub.com/ondoprotocol/usdy-noble/x/aura/types/blocklist
across the codebase.x/aura/keeper/msg_server_blocklist.go (1)
8-8
: Import Path Update: Correctness and ConsistencyThe import path has been updated to reflect the new module structure. Ensure that all references to
github.com/noble-assets/aura/x/aura/types/blocklist
are correctly replaced withgithub.com/ondoprotocol/usdy-noble/x/aura/types/blocklist
across the codebase.x/aura/module.go (1)
15-18
: Import Path Update: Ensure consistency and correctness.The import paths have been updated to reflect the new module path
github.com/ondoprotocol/usdy-noble
. Ensure that these paths are correct and consistent across the entire codebase.Verification successful
Import Path Update Verified:
The import paths have been updated to
github.com/ondoprotocol/usdy-noble
consistently across the codebase. There are no occurrences of the old paths.
- Verified in multiple files including
x/aura/module.go
,x/aura/genesis.go
,utils/mocks/aura.go
, and more.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that the updated import paths are consistent across the codebase. # Test: Search for the old import paths. Expect: No occurrences of the old paths. rg --type go $'github.com/noble-assets/aura' # Test: Search for the new import paths. Expect: Occurrences of the new paths. rg --type go $'github.com/ondoprotocol/usdy-noble'Length of output: 4479
proto/aura/v1/tx.proto (1)
7-7
: Protobuf Go Package Update: Ensure consistency and correctness.The
go_package
option has been updated to reflect the new module pathgithub.com/ondoprotocol/usdy-noble/x/aura/types
. Ensure that this path is correct and consistent across all Protobuf files.Verification successful
Protobuf Go Package Update: Verified
The
go_package
option has been successfully updated togithub.com/ondoprotocol/usdy-noble/x/aura/types
across all relevant Protobuf files. No instances of the old package path were found.
proto/aura/v1/tx.proto
proto/aura/v1/query.proto
proto/aura/v1/genesis.proto
proto/aura/v1/events.proto
proto/aura/blocklist/v1/tx.proto
proto/aura/blocklist/v1/query.proto
proto/aura/blocklist/v1/genesis.proto
proto/aura/blocklist/v1/events.proto
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that the updated `go_package` option is consistent across all Protobuf files. # Test: Search for the old `go_package` option. Expect: No occurrences of the old option. rg --type proto $'github.com/noble-assets/aura' # Test: Search for the new `go_package` option. Expect: Occurrences of the new option. rg --type proto $'github.com/ondoprotocol/usdy-noble'Length of output: 159
Script:
#!/bin/bash # Description: Verify that the updated `go_package` option is consistent across all Protobuf files. # Find all .proto files proto_files=$(fd --extension proto) # Test: Search for the old `go_package` option. Expect: No occurrences of the old option. rg 'github.com/noble-assets/aura' $proto_files # Test: Search for the new `go_package` option. Expect: Occurrences of the new option. rg 'github.com/ondoprotocol/usdy-noble' $proto_filesLength of output: 1713
x/aura/keeper/msg_server_blocklist_test.go (1)
7-10
: Import Path Update: Ensure consistency and correctness.The import paths have been updated to reflect the new module path
github.com/ondoprotocol/usdy-noble
. Ensure that these paths are correct and consistent across the entire codebase.Verification successful
Import Path Update: Verified consistency and correctness.
The import paths have been successfully updated to
github.com/ondoprotocol/usdy-noble
and are consistent across the codebase. No occurrences of the old pathsgithub.com/noble-assets/aura
were found.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that the updated import paths are consistent across the codebase. # Test: Search for the old import paths. Expect: No occurrences of the old paths. rg --type go $'github.com/noble-assets/aura' # Test: Search for the new import paths. Expect: Occurrences of the new paths. rg --type go $'github.com/ondoprotocol/usdy-noble'Length of output: 4479
x/aura/keeper/keeper_test.go (1)
9-11
: Import Path Update: Ensure consistency and correctness.The import paths have been updated to reflect the new module path
github.com/ondoprotocol/usdy-noble
. Ensure that these paths are correct and consistent across the entire codebase.Verification successful
Import Path Update Verified:
The import paths have been updated to
github.com/ondoprotocol/usdy-noble
consistently across the codebase, and no occurrences of the old import pathsgithub.com/noble-assets/aura
were found.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that the updated import paths are consistent across the codebase. # Test: Search for the old import paths. Expect: No occurrences of the old paths. rg --type go $'github.com/noble-assets/aura' # Test: Search for the new import paths. Expect: Occurrences of the new paths. rg --type go $'github.com/ondoprotocol/usdy-noble'Length of output: 4479
e2e/utils.go (3)
10-10
: Update import paths to reflect module restructuring.The import path for the
aura
module has been updated togithub.com/ondoprotocol/usdy-noble/x/aura
.
11-11
: Update import paths to reflect module restructuring.The import path for the
aura/types
module has been updated togithub.com/ondoprotocol/usdy-noble/x/aura/types
.
12-12
: Update import paths to reflect module restructuring.The import path for the
aura/types/blocklist
module has been updated togithub.com/ondoprotocol/usdy-noble/x/aura/types/blocklist
.simapp/go.mod (2)
1-1
: Update module path to reflect rebranding.The module path has been updated to
github.com/ondoprotocol/usdy-noble/simapp
.
161-162
: Update module replacement directive.The module replacement path for
github.com/ondoprotocol/usdy-noble
has been updated to use the local path../
.simapp/simd/cmd/root.go (1)
29-29
: Update import paths to reflect module restructuring.The import path for the
simapp
module has been updated togithub.com/ondoprotocol/usdy-noble/simapp
.e2e/go.mod (2)
1-1
: Update module path to reflect rebranding.The module path has been updated to
github.com/ondoprotocol/usdy-noble/e2e
.
177-178
: Update module replacement directive.The module replacement path for
github.com/ondoprotocol/usdy-noble
has been updated to use the local path../
.x/aura/keeper/msg_server.go (1)
11-11
: Verify the import path update.Ensure that the new import path
github.com/ondoprotocol/usdy-noble/x/aura/types
is correct and consistent with the project restructuring.Verification successful
Import path update verified.
The new import path
github.com/ondoprotocol/usdy-noble/x/aura/types
is used consistently across the codebase, confirming its correctness.
- utils/mocks/aura.go
- utils/mocks/bank.go
- x/aura/module.go
- x/aura/genesis.go
- x/aura/types/genesis.pb.go
- x/aura/types/genesis.go
- x/aura/types/codec.go
- x/aura/keeper/keeper.go
- x/aura/keeper/state.go
- x/aura/keeper/query_server_test.go
- simapp/app.go
- x/aura/keeper/query_server.go
- x/aura/keeper/msg_server_test.go
- x/aura/client/cli/query.go
- x/aura/client/cli/tx.go
- x/aura/keeper/keeper_test.go
- e2e/utils.go
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the new import path is correct and consistent. # Test: Search for the new import path in the codebase. Expect: Consistent usage of the new path. rg --type go $'github.com/ondoprotocol/usdy-noble/x/aura/types'Length of output: 2386
simapp/app.go (1)
61-63
: Verify the import path updates.Ensure that the new import paths
github.com/ondoprotocol/usdy-noble/x/aura
,github.com/ondoprotocol/usdy-noble/x/aura/keeper
, andgithub.com/ondoprotocol/usdy-noble/x/aura/types
are correct and consistent with the project restructuring.Verification successful
Import paths verified successfully.
The new import paths
github.com/ondoprotocol/usdy-noble/x/aura
,github.com/ondoprotocol/usdy-noble/x/aura/keeper
, andgithub.com/ondoprotocol/usdy-noble/x/aura/types
are correct and consistently used across the codebase.
github.com/ondoprotocol/usdy-noble/x/aura
github.com/ondoprotocol/usdy-noble/x/aura/keeper
github.com/ondoprotocol/usdy-noble/x/aura/types
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the new import paths are correct and consistent. # Test: Search for the new import paths in the codebase. Expect: Consistent usage of the new paths. rg --type go $'github.com/ondoprotocol/usdy-noble/x/aura' rg --type go $'github.com/ondoprotocol/usdy-noble/x/aura/keeper' rg --type go $'github.com/ondoprotocol/usdy-noble/x/aura/types'Length of output: 6327
Summary by CodeRabbit
noble-assets
toondoprotocol/usdy-noble
.heighliner-build-action
dependency tov1.0.2
in workflow configurations.clone-key
configuration from workflow files.