Skip to content

Merge pull request #3 from chiefMarlin/master #33

Merge pull request #3 from chiefMarlin/master

Merge pull request #3 from chiefMarlin/master #33

Triggered via push December 8, 2023 23:02
Status Failure
Total duration 43s
Artifacts
Fit to window
Zoom out
Zoom in

Annotations

7 errors and 6 warnings
lint: pkg/asciicast/event_test.go#L34
G601: Implicit memory aliasing in for loop. (gosec)
lint: internal/svg/svg.go#L153
expressions should not be cuddled with blocks (wsl)
lint: internal/svg/svg.go#L29
type must not be placed after const (desired order: type,const,var,func) (decorder)
lint: internal/svg/svg.go#L38
type must not be placed after const (desired order: type,const,var,func) (decorder)
lint: cmd/termsvg/main.go#L22
type must not be placed after var (desired order: type,const,var,func) (decorder)
lint: cmd/termsvg/main.go#L26
type must not be placed after var (desired order: type,const,var,func) (decorder)
lint
issues found
test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, golangci/golangci-lint-action@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
lint: pkg/asciicast/asciicast.go#L147
superfluous-else: if block ends with a continue statement, so drop this else and outdent its block (revive)
lint: internal/svg/svg.go#L188
superfluous-else: if block ends with a continue statement, so drop this else and outdent its block (revive)
lint: cmd/termsvg/main.go#L34
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)