From d42ac64492ab8d7a40e33c3a8f02030ad1cb9717 Mon Sep 17 00:00:00 2001 From: dfawley Date: Tue, 11 Sep 2018 10:33:48 -0700 Subject: [PATCH] move tools.go to sub-package (#2300) `godep` by default pulls in all dependencies regardless of build tags. If `tools.go` is in our main package, that can cause unnecessary dependencies to be pulled into others' repos that otherwise don't need them. Moving it to an unused sub-package appears to fix this problem. --- tools.go => test/tools/tools.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) rename tools.go => test/tools/tools.go (79%) diff --git a/tools.go b/test/tools/tools.go similarity index 79% rename from tools.go rename to test/tools/tools.go index d7ee73bf..f65618b8 100644 --- a/tools.go +++ b/test/tools/tools.go @@ -18,12 +18,12 @@ * */ -// This file exists to cause `go mod` and `go get` to believe these packages -// are dependencies, even though they are not runtime dependencies. This means -// they will appear in our `go.mod` file, but will not be a part of the build -// unless the "tools" build tag is specified. +// This package exists to cause `go mod` and `go get` to believe these tools +// are dependencies, even though they are not runtime dependencies of any grpc +// package. This means they will appear in our `go.mod` file, but will not be +// a part of the build. -package grpc +package tools import ( _ "github.com/client9/misspell/cmd/misspell"