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

Clean up Msgs and Codecs. #1957

Merged
merged 43 commits into from
May 13, 2024
Merged
Changes from 1 commit
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
f80182b
[1760]: Rename all the msg.go files (and their relatives) to msgs.go.
SpicyLemon Apr 25, 2024
2acde49
[1760]: Standardize all the msg and codec definition to use a public …
SpicyLemon Apr 25, 2024
c97e0ba
[1760]: Create RunGetSignersTests to standardize checking signers-rel…
SpicyLemon Apr 25, 2024
3915c4f
[1760]: Switch the exchange GetSigners test over to the new RunGetSig…
SpicyLemon Apr 25, 2024
6084f42
[1760]: Switch the metadata GetSigners test over to the new RunGetSig…
SpicyLemon Apr 25, 2024
55c63e8
[1760]: For the legacy test runner, just make sure the panic contains…
SpicyLemon Apr 25, 2024
ca22068
[1760]: Add GetSigners tests to the attribute module.
SpicyLemon Apr 25, 2024
754144b
[1760]: Streamline the exchange GetSigners maker definitions.
SpicyLemon Apr 25, 2024
1ad68dc
[1760]: Change ExpPanic fields to ExpInErr since the new stuff isn't …
SpicyLemon Apr 25, 2024
cd2c8eb
[1760]: Add a GetSigners tests to the ibchooks module, and fix the Ms…
SpicyLemon Apr 25, 2024
aeb142b
[1760]: Add GetSigners tests to the ibcratelimit module.
SpicyLemon Apr 25, 2024
de5cc29
[1760]: GetSigners Tests on all the marker Msgs, and fix MsgCancelReq…
SpicyLemon Apr 25, 2024
c45b90d
[1760]: Switch to RunGetSignersTests in the msgfees module.
SpicyLemon Apr 25, 2024
11e303e
[1760]: Switch to RunGetSignersTests in the name module.
SpicyLemon Apr 26, 2024
5c7bd21
[1760]: Switch to RunGetSignersTests in the oracle module.
SpicyLemon Apr 26, 2024
6f2829d
[1760]: Switch to RunGetSignersTests for the quarantine module.
SpicyLemon Apr 26, 2024
9586b7b
[1760]: Switch to RunGetSignersTests for the sanction module.
SpicyLemon Apr 26, 2024
297cb5d
[1760]: Switch to RunGetSignersTests for the trigger module.
SpicyLemon Apr 26, 2024
fc26552
[1760]: Get rid of most of the GetSigners methods (the ones that were…
SpicyLemon Apr 26, 2024
ff4bac0
[1760]: Remove the marker module's GetSigners methods, and update the…
SpicyLemon Apr 26, 2024
91bf74a
[1760]: Remove the GetSigners() methods from the metadata module.
SpicyLemon Apr 26, 2024
8bfd5d1
[1760]: Get rid of some ibchooks msg methods that used to be required…
SpicyLemon Apr 26, 2024
f68ee9b
[1760]: Reorder the stuff in marker/types/msgs.go to match the order …
SpicyLemon Apr 26, 2024
32e5701
[1760]: Reorder the stuff in names msgs.go to match the proto ordering.
SpicyLemon Apr 26, 2024
eda386a
[1760]: Reorganize the metadata msgs.go stuff.
SpicyLemon Apr 26, 2024
1631742
[1760]: Clean up the attribute and ibchooks protos a little.
SpicyLemon Apr 26, 2024
f9c9574
[1760]: Get rid of most of the yaml tags from the protos. Only ones l…
SpicyLemon Apr 26, 2024
0591d78
Merge branch 'main' into dwedul/1760-get-signers
SpicyLemon May 6, 2024
04b6482
[1760]: Remove the rest of the uses of gogoproto.moretags.
SpicyLemon May 6, 2024
8df46bd
[1760]: Clean up some unneeded gogoproto.goproto_stringer options fro…
SpicyLemon May 6, 2024
98bf7a0
[1760]: Update all the Coins fields in the protos to match the same o…
SpicyLemon May 6, 2024
3c6c635
[1760]: Remove all uses of any ModuleCdc variables, and delete them all.
SpicyLemon May 6, 2024
4da9f03
[1760]: In GetFeePayerUsingFeeGrant, cast the feePayer and feeGranter…
SpicyLemon May 6, 2024
7d40bf5
[1760]: Get rid of all the String() methods that just do yaml convers…
SpicyLemon May 7, 2024
06aaf93
[1760]: Refactor the MetadataAddress.Format method to handle more cas…
SpicyLemon May 7, 2024
10694a8
Merge branch 'main' into dwedul/1760-get-signers
SpicyLemon May 7, 2024
f56953a
[1760]: Remove a private unused function: stringsToAccAddresses
SpicyLemon May 7, 2024
7c14061
[1760]: Switch to sigs.k8s.io/yaml (from gopkg.in/yaml.v2) in the las…
SpicyLemon May 7, 2024
7208658
[1760]: Move DefineCustomGetSigners to the top of the file.
SpicyLemon May 7, 2024
c582371
[1760]: Add changelog entries.
SpicyLemon May 7, 2024
378c796
[1760]: Move now-indirect require of gopkg.in/yaml.v2 to the indirect…
SpicyLemon May 8, 2024
cd11009
[1760]: Tweak the MetadataAddress.Format method: add some comments an…
SpicyLemon May 8, 2024
004bc00
Merge branch 'main' into dwedul/1760-get-signers
SpicyLemon May 13, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[1760]: Move now-indirect require of gopkg.in/yaml.v2 to the indirect…
… section.
SpicyLemon committed May 8, 2024
commit 378c7963d0f885a9ee12e1859c07e919ce0f39ee
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
@@ -44,7 +44,6 @@ require (
google.golang.org/genproto/googleapis/api v0.0.0-20240227224415-6ceb2ff114de
google.golang.org/grpc v1.63.2
google.golang.org/protobuf v1.34.1
gopkg.in/yaml.v2 v2.4.0 // indirect
sigs.k8s.io/yaml v1.4.0
)

@@ -202,6 +201,7 @@ require (
google.golang.org/genproto v0.0.0-20240227224415-6ceb2ff114de // indirect
google.golang.org/genproto/googleapis/rpc v0.0.0-20240401170217-c3f982113cda // indirect
gopkg.in/ini.v1 v1.67.0 // indirect
gopkg.in/yaml.v2 v2.4.0 // indirect
gopkg.in/yaml.v3 v3.0.1 // indirect
gotest.tools/v3 v3.5.1 // indirect
nhooyr.io/websocket v1.8.10 // indirect

Unchanged files with check annotations Beta

// ProposalContents returns all the exchange content functions used to
// simulate governance proposals, of which there are none for the exchange module.
func (am AppModule) ProposalContents(_ module.SimulationState) []simtypes.WeightedProposalContent {

Check failure on line 139 in x/exchange/module/module.go

GitHub Actions / golangci-lint

SA1019: simtypes.WeightedProposalContent is deprecated: Use WeightedProposalMsg instead. (staticcheck)
return nil
}
}
seen[addr] = struct{}{}
}
if err := f.Coins.Validate(); err != nil {

Check failure on line 88 in x/quarantine/quarantine.go

GitHub Actions / golangci-lint

if-return: redundant if ...; err != nil check, just return error instead. (revive)
return err
}
return nil
)
// ProposalContents defines the module weighted proposals' contents
func ProposalContents(k keeper.Keeper) []simtypes.WeightedProposalContent {

Check failure on line 36 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.WeightedProposalContent is deprecated: Use WeightedProposalMsg instead. (staticcheck)
return []simtypes.WeightedProposalContent{

Check failure on line 37 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.WeightedProposalContent is deprecated: Use WeightedProposalMsg instead. (staticcheck)
simulation.NewWeightedProposalContent(
OpWeightSupplyIncreaseProposal,
simappparams.DefaultWeightSupplyIncreaseProposalContent,
}
// SimulateCreateSupplyIncreaseProposalContent generates random increase marker supply proposal content
func SimulateCreateSupplyIncreaseProposalContent(k keeper.Keeper) simtypes.ContentSimulatorFn {

Check failure on line 72 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.ContentSimulatorFn is deprecated: Use MsgSimulatorFn instead. (staticcheck)
return func(r *rand.Rand, ctx sdk.Context, accs []simtypes.Account) simtypes.Content {

Check failure on line 73 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.Content is deprecated: Use MsgSimulatorFn instead. (staticcheck)
dest := ""
if r.Intn(100) < 40 {
acc, _ := simtypes.RandomAcc(r, accs)
}
// SimulateCreateSupplyDecreaseProposalContent generates random create-root-name proposal content
func SimulateCreateSupplyDecreaseProposalContent(k keeper.Keeper) simtypes.ContentSimulatorFn {

Check failure on line 103 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.ContentSimulatorFn is deprecated: Use MsgSimulatorFn instead. (staticcheck)
return func(r *rand.Rand, ctx sdk.Context, _ []simtypes.Account) simtypes.Content {

Check failure on line 104 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.Content is deprecated: Use MsgSimulatorFn instead. (staticcheck)
m := randomGovMarkerWithMaxStatus(r, ctx, k, types.StatusActive)
if m == nil {
return nil
}
// SimulateCreateSetAdministratorProposalContent generates random create-root-name proposal content
func SimulateCreateSetAdministratorProposalContent(k keeper.Keeper) simtypes.ContentSimulatorFn {

Check failure on line 123 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.ContentSimulatorFn is deprecated: Use MsgSimulatorFn instead. (staticcheck)
return func(r *rand.Rand, ctx sdk.Context, accs []simtypes.Account) simtypes.Content {

Check failure on line 124 in x/marker/simulation/proposals.go

GitHub Actions / golangci-lint

SA1019: simtypes.Content is deprecated: Use MsgSimulatorFn instead. (staticcheck)
m := randomGovMarkerWithMaxStatus(r, ctx, k, types.StatusDestroyed)
if m == nil {
return nil
syntax = "proto3";
package provenance.ibchooks.v1;
import "gogoproto/gogo.proto";

Check failure on line 4 in proto/provenance/ibchooks/v1/params.proto

GitHub Actions / Protobuf Checks

Import "gogoproto/gogo.proto" is unused.
option go_package = "github.com/provenance-io/provenance/x/ibchooks/types";
package provenance.ibchooks.v1;
import "cosmos/msg/v1/msg.proto";
import "gogoproto/gogo.proto";

Check failure on line 5 in proto/provenance/ibchooks/v1/tx.proto

GitHub Actions / Protobuf Checks

Import "gogoproto/gogo.proto" is unused.
option go_package = "github.com/provenance-io/provenance/x/ibchooks/types";
syntax = "proto3";
package provenance.ibcratelimit.v1;
import "gogoproto/gogo.proto";

Check failure on line 4 in proto/provenance/ibcratelimit/v1/params.proto

GitHub Actions / Protobuf Checks

Import "gogoproto/gogo.proto" is unused.
option go_package = "github.com/provenance-io/provenance/x/ibcratelimit";
option java_package = "io.provenance.ibcratelimit.v1";