-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
feat(runtime/v2): store loader on simappv2 #21704
Changes from 52 commits
ebf5006
268c034
babf339
6780043
61c0343
b8ac0e5
a1b0a5f
71d0027
cadccd4
7bc73f0
728313a
e97fda6
57c9751
a32d778
4bcf345
7006da5
91570fb
de93866
9fc2546
7a1fa60
6b32d53
db542cf
70f1090
ea0193d
c552beb
f67770e
2948e09
a0ffd0a
519d6b7
f199f06
11ec8d2
a634633
e0093cf
5ab7c1a
da07a4a
264434e
aedc7b0
8f97b21
9156b86
858af16
a789413
dee4c3b
5051c67
73e839b
98b4397
4324f42
3ea6367
1874e0c
64eb9a4
8035440
1aeda3a
346e715
7586ee2
4ecf936
c732286
b2082c8
8c23aa5
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,7 @@ import ( | |
"github.com/spf13/viper" | ||
"github.com/stretchr/testify/require" | ||
|
||
app2 "cosmossdk.io/core/app" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ? |
||
"cosmossdk.io/core/comet" | ||
context2 "cosmossdk.io/core/context" | ||
"cosmossdk.io/core/server" | ||
|
@@ -21,6 +22,7 @@ import ( | |
serverv2 "cosmossdk.io/server/v2" | ||
comettypes "cosmossdk.io/server/v2/cometbft/types" | ||
"cosmossdk.io/store/v2/db" | ||
authtypes "cosmossdk.io/x/auth/types" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. probably from some of my tests, will remove |
||
banktypes "cosmossdk.io/x/bank/types" | ||
|
||
"github.com/cosmos/cosmos-sdk/crypto/keys/secp256k1" | ||
|
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.
can we add a go doc?
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.
Given that it is in runtime/v2, no need to prepend it by
V2
imho.For consistency, shall we place the other one from x/upgrade in runtime?
And just alias it back in x/upgrade for not introducing breaking changes?
Additionally, can we add a
:tip:
in x/upgrade docs mentioning for server/v2 this helper is in runtime/v2There 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.
agreed