Skip to content
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: add autocli service proto #13597

Merged
merged 12 commits into from
Oct 25, 2022
Prev Previous commit
Next Next commit
address review comments
aaronc committed Oct 24, 2022
commit 8f5e3e65832947c0b00cd7ec7dbe58593ff909c7
Original file line number Diff line number Diff line change
@@ -12,6 +12,8 @@ option go_package = "cosmossdk.io/api/cosmos/base/cli/v1;cliv1";
service Query {
// AppOptions returns the autocli options for all of the modules in an app.
rpc AppOptions(AppOptionsRequest) returns (AppOptionsResponse) {
// NOTE: autocli options SHOULD NOT be part of consensus and module_query_safe
// should be kept as false.
option (cosmos.query.v1.module_query_safe) = false;
}
}
2 changes: 2 additions & 0 deletions proto/cosmos/reflection/v1/reflection.proto
Original file line number Diff line number Diff line change
@@ -12,6 +12,8 @@ service ReflectionService {
// FileDescriptors queries all the file descriptors in the app in order
// to enable easier generation of dynamic clients.
rpc FileDescriptors(FileDescriptorsRequest) returns (FileDescriptorsResponse) {
// NOTE: file descriptors SHOULD NOT be part of consensus because they
// include changes to doc commands and module_query_safe should be kept as false.
option (cosmos.query.v1.module_query_safe) = false;
}
}