uber-go / goleak

Goroutine leak detector
MIT License
4.48k stars 148 forks source link

ci: Use golangci-lint for linting #108

Closed abhinav closed 10 months ago

abhinav commented 10 months ago

Instead of hand-managing and running linters, use golangci-lint. This simplifies the Makefile and allows lint to run in parallel with the build/test job.

Along with the golangci-lint defaults, enable a couple other linters we generally agree with. See also uber-go/zap#1323 and uber-go/sally#121 for similar changes.

codecov[bot] commented 10 months ago

Codecov Report

Merging #108 (586e0e3) into master (03cb6e6) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #108   +/-   ##
=======================================
  Coverage   96.58%   96.58%           
=======================================
  Files           5        5           
  Lines         234      234           
=======================================
  Hits          226      226           
  Misses          5        5           
  Partials        3        3           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more