Exclude file permissions line from gosec linting #474
check-commit-message-push.yml
on: push
Check commit message style on push
4s
Annotations
16 errors and 2 warnings
Check commit message style on push
The message 1 is invalid:
* The line 4 of the message (line 2 of the body) exceeds the limit of 72 characters. The line contains 89 characters: "[by default on file permissions above 0600](https://github.com/securego/gosec/issues/107)". Please reformat the body so that all the lines fit 72 characters.
The original message was:
Exclude file permissions line from gosec linting
The [gosec linter](https://github.com/securego/gosec) warns
[by default on file permissions above 0600](https://github.com/securego/gosec/issues/107)
We need the permissions to be 0644 for this line (because it has to be
written to), so we exclude it from linting.
|
[golangci] internal/slice/slice.go#L21:
internal/slice/slice.go#L21
exported: exported function StartsWithAnyOf should have comment or be unexported (revive)
|
[golangci] internal/github/action.go#L82:
internal/github/action.go#L82
type `specified` is unused (unused)
|
[golangci] internal/github/pullrequest/labels.go#L7:
internal/github/pullrequest/labels.go#L7
import 'github.com/agilepathway/label-checker/internal/error/panic' is not allowed from list 'Main' (depguard)
|
[golangci] internal/github/pullrequest/labels.go#L8:
internal/github/pullrequest/labels.go#L8
import 'github.com/agilepathway/label-checker/internal/slice' is not allowed from list 'Main' (depguard)
|
[golangci] internal/github/pullrequest/pull_request.go#L9:
internal/github/pullrequest/pull_request.go#L9
import 'github.com/agilepathway/label-checker/internal/error/panic' is not allowed from list 'Main' (depguard)
|
[golangci] internal/github/pullrequest/pull_request.go#L10:
internal/github/pullrequest/pull_request.go#L10
import 'github.com/shurcooL/githubv4' is not allowed from list 'Main' (depguard)
|
[golangci] label_checker.go#L7:
label_checker.go#L7
import 'github.com/agilepathway/label-checker/internal/github' is not allowed from list 'Main' (depguard)
|
[golangci] internal/github/action.go#L16:
internal/github/action.go#L16
import 'github.com/agilepathway/label-checker/internal/error/panic' is not allowed from list 'Main' (depguard)
|
[golangci] internal/github/action.go#L17:
internal/github/action.go#L17
import 'github.com/agilepathway/label-checker/internal/github/pullrequest' is not allowed from list 'Main' (depguard)
|
[golangci] internal/github/action.go#L130:
internal/github/action.go#L130
mnd: Magic number: 0o644, in <argument> detected (gomnd)
|
[golangci] internal/github/action.go#L11:
internal/github/action.go#L11
SA1019: "io/ioutil" has been deprecated since Go 1.19: As of Go 1.16, the same functionality is now provided by package [io] or package [os], and those implementations should be preferred in new code. See the specific function documentation for details. (staticcheck)
|
[golangci] internal/github/pullrequest/labels.go#L32:
internal/github/pullrequest/labels.go#L32
return statements should not be cuddled if block has more than two lines (wsl)
|
[golangci] internal/github/action.go#L133:
internal/github/action.go#L133
only one cuddle assignment allowed before if statement (wsl)
|
[golangci] internal/github/action_test.go#L158:
internal/github/action_test.go#L158
expressions should not be cuddled with blocks (wsl)
|
[golangci] internal/github/action_test.go#L272:
internal/github/action_test.go#L272
return statements should not be cuddled if block has more than two lines (wsl)
|
Check commit message style on push
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: mristin/[email protected]. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Check commit message style on push
The following actions uses node12 which is deprecated and will be forced to run on node16: mristin/[email protected]. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|