Skip to content

Merge pull request #178 from penny-vault/dependabot/go_modules/github… #375

Merge pull request #178 from penny-vault/dependabot/go_modules/github…

Merge pull request #178 from penny-vault/dependabot/go_modules/github… #375

Triggered via push March 25, 2024 06:21
Status Failure
Total duration 2m 34s
Artifacts

lint.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 8 warnings
lint: errorcode/errors.go#L1
Pattern ((20\d\d\-2024)|(2024)) doesn't match. (goheader)
lint: fidelity/accounts.go#L1
Pattern ((20\d\d\-2024)|(2024)) doesn't match. (goheader)
lint: fidelity/activity.go#L1
Pattern ((20\d\d\-2024)|(2024)) doesn't match. (goheader)
lint: fidelity/tickers.go#L51
err113: do not define dynamic errors, use wrapped static errors instead: "errors.New(\"could not find locator: `body > table > tbody > tr > td:nth-child(2) > table:nth-child(4) > tbody > tr > td:nth-child(2) > table > tbody > tr:nth-child(3) > td:nth-child(1) > font`\")" (goerr113)
lint: fidelity/tickers.go#L69
err113: do not define dynamic errors, use wrapped static errors instead: "errors.New(\"could not find locator: `body > table > tbody > tr > td:nth-child(2) > table:nth-child(4) > tbody > tr > td:nth-child(2) > table > tbody > tr:nth-child(3) > td:nth-child(3) > font`\")" (goerr113)
lint: fidelity/playwright.go#L52
SA1019: page.Type is deprecated: In most cases, you should use [Locator.Fill] instead. You only need to press keys one by one if there is special keyboard handling on the page - in this case use [Locator.PressSequentially]. (staticcheck)
lint: fidelity/playwright.go#L57
SA1019: page.Type is deprecated: In most cases, you should use [Locator.Fill] instead. You only need to press keys one by one if there is special keyboard handling on the page - in this case use [Locator.PressSequentially]. (staticcheck)
lint: fidelity/playwright.go#L61
SA1019: page.Click is deprecated: Use locator-based [Locator.Click] instead. Read more about [locators]. (staticcheck)
lint: fidelity/playwright.go#L66
SA1019: page.WaitForTimeout is deprecated: Never wait for timeout in production. Tests that wait for time are inherently flaky. Use [Locator] actions and web assertions that wait automatically. (staticcheck)
lint: fidelity/playwright.go#L95
must be dispatched by Msg or Send method (zerologlint)
lint: fidelity/accounts.go#L70
unused-parameter: parameter 'key' seems to be unused, consider removing or renaming it as _ (revive)
lint: fidelity/activity.go#L198
unused-parameter: parameter 'key' seems to be unused, consider removing or renaming it as _ (revive)
lint: cmd/encrypt.go#L59
unused-parameter: parameter 'cmd' seems to be unused, consider removing or renaming it as _ (revive)
lint: cmd/activity.go#L61
unused-parameter: parameter 'cmd' seems to be unused, consider removing or renaming it as _ (revive)
lint: cmd/login.go#L38
unused-parameter: parameter 'cmd' seems to be unused, consider removing or renaming it as _ (revive)
lint: fidelity/fidelity_suite_test.go#L21
dot-imports: should not use dot imports (revive)
lint: fidelity/fidelity_suite_test.go#L22
dot-imports: should not use dot imports (revive)
lint: fidelity/activity_test.go#L22
dot-imports: should not use dot imports (revive)