Avoid providing default home dir to depinject
All checks were successful
Unit Tests / test-unit (pull_request) Successful in 1m57s
SDK Tests / sdk_tests_nameservice_expiry (pull_request) Successful in 7m42s
SDK Tests / sdk_tests (pull_request) Successful in 8m10s
SDK Tests / sdk_tests_auctions (pull_request) Successful in 13m0s
Integration Tests / test-integration (pull_request) Successful in 2m36s
Build / build (pull_request) Successful in 2m38s
E2E Tests / test-e2e (pull_request) Successful in 3m57s
Lint / Run golangci-lint (pull_request) Successful in 5m36s

This commit is contained in:
Prathamesh Musale 2024-07-09 16:21:22 +05:30
parent d19e833d7f
commit 8750ef06f0

View File

@ -134,7 +134,9 @@ func ProvideClientContext(
WithLegacyAmino(legacyAmino).
WithInput(os.Stdin).
WithAccountRetriever(types.AccountRetriever{}).
WithHomeDir(app.DefaultNodeHome).
// Workaround: Avoid providing DefaultNodeHome to depinject as it is given precedence over the one passed using --home flag in some CLI commands
// TODO: Implement proper fix
// WithHomeDir(app.DefaultNodeHome).
WithViper(EnvPrefix) // env variable prefix
// Read the config again to overwrite the default values with the values from the config file