DataDog / dd-trace-go

Datadog Go Library including APM tracing, profiling, and security monitoring.
https://docs.datadoghq.com/tracing/
Other
657 stars 438 forks source link

proposal: Improve CI linting #758

Open knusbaum opened 4 years ago

knusbaum commented 4 years ago

Based on #753 we need to do better checking of the source to catch issues with other GOOS/GOARCH.

@aeneasr suggested linting would help with this.

A quick look shows that the gometalinter has been deprecated in favor of golangci-lint. We should see if/how we can incorporate golangci-lint into the CI to catch these types of issues.

knusbaum commented 1 year ago

@ajgajg1134 suggested we can at least run GOOS=X GOARCH=Y go build ./... for every arch/os pair to ensure compile time errors are not present.

May be easier than configuring linters.