Skip to content

Preparation for subscribing with multiple accounts and test that confirms current issues #6988

Preparation for subscribing with multiple accounts and test that confirms current issues

Preparation for subscribing with multiple accounts and test that confirms current issues #6988

Triggered via pull request September 19, 2023 09:32
Status Failure
Total duration 1m 4s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

golangci-lint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

8 errors and 2 warnings
lint: tools/walletextension/accountmanager/account_manager.go#L69
Error return value of `m.subscriptionsManager.HandleNewSubscriptions` is not checked (errcheck)
lint: tools/walletextension/common/common.go#L8
File is not `gofumpt`-ed (gofumpt)
lint: tools/walletextension/common/common.go#L11
File is not `gofumpt`-ed (gofumpt)
lint: tools/walletextension/subscriptions/subscriptions.go#L5
File is not `gofumpt`-ed (gofumpt)
lint: tools/walletextension/accountmanager/account_manager.go#L11
File is not `goimports`-ed (goimports)
lint: tools/walletextension/subscriptions/subscriptions.go#L35
Consider pre-allocating `subscriptions` (prealloc)
lint: tools/walletextension/subscriptions/subscriptions.go#L55
S1000: should use for range instead of for { select {} } (gosimple)
lint
issues found
lint
Restore cache failed: Dependencies file is not found in /home/runner/work/go-obscuro/go-obscuro. Supported file pattern: go.sum
lint: tools/walletextension/subscriptions/subscriptions.go#L26
unused-parameter: parameter 'resp' seems to be unused, consider removing or renaming it as _ (revive)