WireGuard / wgctrl-go

Package wgctrl enables control of WireGuard interfaces on multiple platforms.
https://godoc.org/golang.zx2c4.com/wireguard/wgctrl
MIT License
727 stars 85 forks source link

Drop staticcheck (honnef.co/go/tools) from go.mod require list #118

Closed stv0g closed 2 years ago

stv0g commented 2 years ago

I realized that adding wgctrl as a dependency to my go package pulls in honnef.co/go/tools and as transitive dependency.

From my understanding this package is only used to pin a specific staticcheck version.

I find it quite unconventional to use the go.mod file for this purpose. At least I haven't seen in any other go package so far.

Can we remove this dependency alongside the tools.go file?

mdlayher commented 2 years ago

Yep, this is something I had tried out briefly but have been removing from other projects. Will do.

stv0g commented 2 years ago

Thanks 👍