Preparation for subscribing with multiple accounts and test that confirms current issues #6976
Triggered via pull request
September 18, 2023 15:21
zkokelj
Status
Failure
Total duration
1m 28s
Artifacts
–
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
7 errors and 3 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#L4
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#L37
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#L23
unused-parameter: parameter 'resp' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
tools/walletextension/subscriptions/subscriptions.go#L74
unused-parameter: parameter 'commonChannel' seems to be unused, consider removing or renaming it as _ (revive)
|