Skip to content

chore: enable copyloopvar linter #4160

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.

Sign up for GitHub

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

Merged
julienrbrt merged 21 commits into from
Jun 18, 2024
Merged

chore: enable copyloopvar linter #4160

Show file tree
Hide file tree
Changes from 20 commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
b64a150
copyloopvar
faddat May 19, 2024
fea5c84
enable copyloopvar linter
faddat May 19, 2024
8520870
Update types_command_test.go
faddat May 19, 2024
8b9500a
Merge branch 'main' into faddat/copyloopvar2
faddat May 19, 2024
8c874ea
bump linter
faddat May 19, 2024
f1741c1
Merge branch 'faddat/copyloopvar2' of https://github.com/faddat/starp…
faddat May 19, 2024
f021fe0
update linter settings
faddat May 19, 2024
f386769
cleanup
faddat May 19, 2024
bb4686e
revert mistaken changes
faddat May 19, 2024
5c6c6d5
Merge branch 'main' into faddat/copyloopvar2
faddat May 19, 2024
9af1bf9
Merge remote-tracking branch 'origin/main' into faddat/copyloopvar2
faddat May 21, 2024
b504975
Merge branch 'main' into faddat/copyloopvar2
faddat May 21, 2024
e8c2740
Merge branch 'main' into faddat/copyloopvar2
Pantani May 24, 2024
3666e09
Merge branch 'main' into faddat/copyloopvar2
Pantani May 28, 2024
d2a9c45
Merge branch 'main' into faddat/copyloopvar2
Pantani May 28, 2024
83980e7
Merge branch 'main' into faddat/copyloopvar2
Pantani May 29, 2024
5d46d94
Merge branch 'main' into faddat/copyloopvar2
Pantani Jun 13, 2024
41e3960
rollback chain/init.go changes
Pantani Jun 13, 2024
3d8b580
Merge branch 'main' into faddat/copyloopvar2
julienrbrt Jun 18, 2024
d40ded9
Merge branch 'main' into faddat/copyloopvar2
julienrbrt Jun 18, 2024
901f8f0
Merge branch 'main' into faddat/copyloopvar2
julienrbrt Jun 18, 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
3 changes: 1 addition & 2 deletions .golangci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ linters:
disable-all: true
enable:
- bodyclose
- copyloopvar
- decorder
- depguard
- dogsled
Expand Down Expand Up @@ -70,7 +71,5 @@ linters-settings:
issues:
exclude-dirs:
- ignite/ui
# # timeout for analysis, e.g. 30s, 5m, default is 1m
# timeout: 5m
max-issues-per-linter: 0
max-same-issues: 0
1 change: 1 addition & 0 deletions changelog.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@

### Changes

- [#4160](https://github.com/ignite/cli/pull/4160) Enable copyloopvar linter
- [#4159](https://github.com/ignite/cli/pull/4159) Enable gci linter
- [#4157](https://github.com/ignite/cli/pull/4157) Upgrade golang to 1.22
- [#4094](https://github.com/ignite/cli/pull/4094) Scaffolding a multi-index map using `ignite s map foo bar baz --index foobar,foobaz` is no longer supported. Use one index instead of use `collections.IndexedMap`.
Expand Down
4 changes: 2 additions & 2 deletions go.mod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ module github.com/ignite/cli/v29

go 1.22

toolchain go1.22.2
toolchain go1.22.3

replace (
github.com/99designs/keyring => github.com/cosmos/keyring v1.2.0
Expand Down Expand Up @@ -47,6 +47,7 @@ require (
github.com/goccy/go-yaml v1.11.3
github.com/golangci/golangci-lint v1.57.2
github.com/google/go-github/v48 v48.2.0
github.com/google/go-querystring v1.1.0
github.com/gorilla/mux v1.8.1
github.com/hashicorp/go-hclog v1.6.3
github.com/hashicorp/go-plugin v1.6.0
Expand Down Expand Up @@ -270,7 +271,6 @@ require (
github.com/google/go-cmp v0.6.0 // indirect
github.com/google/go-containerregistry v0.19.1 // indirect
github.com/google/go-dap v0.11.0 // indirect
github.com/google/go-querystring v1.1.0 // indirect
github.com/google/pprof v0.0.0-20240509144519-723abb6459b7 // indirect
github.com/google/uuid v1.6.0 // indirect
github.com/gordonklaus/ineffassign v0.1.0 // indirect
Expand Down
2 changes: 1 addition & 1 deletion ignite/cmd/chain.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -288,7 +288,7 @@ func configMigrationPreRunHandler(cmd *cobra.Command, session *cliui.Session, ap
return err
}

if err := os.WriteFile(cfgPath, buf.Bytes(), 0o755); err != nil {
if err := os.WriteFile(cfgPath, buf.Bytes(), 0o600); err != nil {
return errors.Errorf("config file migration failed: %w", err)
}
}
Expand Down
2 changes: 1 addition & 1 deletion ignite/cmd/plugin_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ func TestLinkPluginCmds(t *testing.T) {
)

// helper to assert pluginInterface.Execute() calls
expectExecute := func(t *testing.T, ctx context.Context, p *mocks.PluginInterface, cmd *plugin.Command) {
expectExecute := func(t *testing.T, _ context.Context, p *mocks.PluginInterface, cmd *plugin.Command) {
t.Helper()
p.EXPECT().
Execute(
Expand Down
2 changes: 1 addition & 1 deletion ignite/internal/plugin/testdata/execute_fail/go.mod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ require (
github.com/cockroachdb/logtags v0.0.0-20230118201751-21c54148d20b // indirect
github.com/cockroachdb/redact v1.1.5 // indirect
github.com/cosmos/btcutil v1.0.5 // indirect
github.com/cosmos/cosmos-sdk v0.50.6 // indirect
github.com/cosmos/cosmos-sdk v0.50.7 // indirect
github.com/cyphar/filepath-securejoin v0.2.4 // indirect
github.com/emirpasic/gods v1.18.1 // indirect
github.com/fatih/color v1.16.0 // indirect
Expand Down
2 changes: 1 addition & 1 deletion ignite/internal/plugin/testdata/execute_ok/go.mod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ require (
github.com/cockroachdb/logtags v0.0.0-20230118201751-21c54148d20b // indirect
github.com/cockroachdb/redact v1.1.5 // indirect
github.com/cosmos/btcutil v1.0.5 // indirect
github.com/cosmos/cosmos-sdk v0.50.6 // indirect
github.com/cosmos/cosmos-sdk v0.50.7 // indirect
github.com/cyphar/filepath-securejoin v0.2.4 // indirect
github.com/emirpasic/gods v1.18.1 // indirect
github.com/fatih/color v1.16.0 // indirect
Expand Down
1 change: 0 additions & 1 deletion ignite/pkg/cmdrunner/cmdrunner.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,6 @@ func (r *Runner) Run(ctx context.Context, steps ...*step.Step) error {
step.Exec.Command,
strings.Join(step.Exec.Args, " "))
}
step := step
if err := ctx.Err(); err != nil {
return err
}
Expand Down
2 changes: 0 additions & 2 deletions ignite/pkg/cosmosgen/generate_composables.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,6 @@ func (g *composablesGenerator) generateComposableTemplates(p generatePayload) er
gg := &errgroup.Group{}

for _, m := range p.Modules {
m := m

gg.Go(func() error {
return g.generateComposableTemplate(m, p)
})
Expand Down
2 changes: 0 additions & 2 deletions ignite/pkg/cosmosgen/generate_typescript.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,6 @@ func (g *tsGenerator) generateModuleTemplates(ctx context.Context) error {
dirCache := cache.New[[]byte](g.g.cacheStorage, dirchangeCacheNamespace)
add := func(sourcePath string, modules []module.Module) {
for _, m := range modules {
m := m

gg.Go(func() error {
cacheKey := m.Pkg.Path
paths := []string{m.Pkg.Path, g.g.opts.jsOut(m)}
Expand Down
1 change: 0 additions & 1 deletion ignite/version/version_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ func TestAssertSupportedCosmosSDKVersion(t *testing.T) {
}

for _, tc := range testCases {
tc := tc
t.Run(tc.name, func(t *testing.T) {
err := version.AssertSupportedCosmosSDKVersion(tc.version)
if tc.errMsg == "" {
Expand Down
2 changes: 1 addition & 1 deletion integration/plugin/testdata/example-plugin/go.mod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ require (
github.com/cockroachdb/logtags v0.0.0-20230118201751-21c54148d20b // indirect
github.com/cockroachdb/redact v1.1.5 // indirect
github.com/cosmos/btcutil v1.0.5 // indirect
github.com/cosmos/cosmos-sdk v0.50.6 // indirect
github.com/cosmos/cosmos-sdk v0.50.7 // indirect
github.com/cyphar/filepath-securejoin v0.2.4 // indirect
github.com/emirpasic/gods v1.18.1 // indirect
github.com/fatih/color v1.16.0 // indirect
Expand Down