Skip to content

Commit

Permalink
Add tenv linter (open-telemetry#5524)
Browse files Browse the repository at this point in the history
This starts adding more linters we aren't currently using, based on the
ones used by [the
collector](https://github.com/open-telemetry/opentelemetry-collector/blob/main/.golangci.yml#L122-L139).

This linter does not currently have any issues, so it requires no
further changes.
I'll be opening other PRs for linters that do require fixes.

* [tenv](https://github.com/sivchari/tenv) requires using `t.Setenv`
rather than `os.Setenv` in tests.

---------

Co-authored-by: Sam Xie <sam@samxie.me>
  • Loading branch information
2 people authored and OrHayat committed Jun 23, 2024
1 parent e85d22e commit a772ccd
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ linters:
- misspell
- revive
- staticcheck
- tenv
- typecheck
- unused

Expand Down

0 comments on commit a772ccd

Please sign in to comment.