| Commit message (Collapse) | Author | Age | Files | |
|---|---|---|---|---|
| * | ci: Use golangci-lint for linting (#121) | Abhinav Gupta | 2023-10-21 | 1 |
| | | | | | | | | | | | | | | Instead of hand-managing and running linters, use golangci-lint. Along with the golangci-lint defaults, enable a couple other linters we generally agree with. See also uber-go/zap#1323 for a similar change. As a result of this, we can: - Drop the dependabot for tools - Run the lint job in parallel with build/test - Simplify the Makefile | |||
| * | Makefile: Simplify (#62) | Abhinav Gupta | 2022-12-18 | 1 |
| This simplifies the Makefile significantly, borrowing patterns we've used in other projects. Namely: - Set GOBIN to a bin subdirectory so that we can `go install` dependencies into it. - Use a shared TEST_FLAGS for `make test` and `make cover`. Without this, we're not running with data race detection in CI. - Build lint step out of separate golint and staticcheck steps. In the future, a gofmt step may also be added. - Move tools dependencies into an unpublished subpackage. Note: I didn't mess with the 'clean' and 'run' targets at the bottom of the file even though they're not necessary to avoid a merge conflict with #60. | ||||
 |
index : sally | |
| a fork with reasonable build of uber-go/sally | grumps |
| aboutsummaryrefslogtreecommitdiff |