-
-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bug report: miss makezero in slice init #84
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
no, I will write more linter's yaml config like this https://github.com/alingse/go-linter-runner/blob/main/jobs/alingse-makezero.yaml#L4 name: alingse-makezero
linter: makezero
workdir: .
install: go install github.com/alingse/makezero@f6a823578e89de5cdfdfef50d4a5d9a09ade16dd
includes:
- non-zero initialized length
excludes:
- called by funcs
- assigned by index this config will install and run the specified linter for repos, and check the output must have and must not have some text. the source repo I run is in here https://github.com/alingse/go-linter-runner/blob/main/source/repos.url.top.1k your repo is in the top3k. The reason that why I create the repo is because I have some idea about linter alingse/sundrylint#2 , I want to check the idea in real-world and than add the useful one to golangci-lint 🤔 and I want to add some auto-report-issuse to process the github actions output. |
I was running github actions to run linter makezero for top github golang repos.
see issues alingse/go-linter-runner#1
and the github actions output https://github.com/alingse/go-linter-runner/actions/runs/9243004504/job/25426564828
https://github.com/montanaflynn/stats/blob/master/mode.go#L16C1-L23
maybe the
mode = make([]float64, 5)
should bemode = make([]float64, 0, 5)
The text was updated successfully, but these errors were encountered: