Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

Commit

Permalink
Use golang v1.13 (#68)
Browse files Browse the repository at this point in the history
* Bump to golang v1.13.0
* Remove TestStaticCheck as staticcheck doesn't respect go cache location
  • Loading branch information
rfay authored Sep 12, 2019
1 parent 41e767c commit c0092fa
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 15 deletions.
2 changes: 1 addition & 1 deletion makefile_components/base_build_go.mak
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ GOFILES = $(shell find $(SRC_DIRS) -name "*.go")

BUILD_OS = $(shell go env GOHOSTOS)

BUILD_IMAGE ?= drud/golang-build-container:v1.12.1
BUILD_IMAGE ?= drud/golang-build-container:v1.13.0

BUILD_BASE_DIR ?= $(PWD)

Expand Down
2 changes: 2 additions & 0 deletions tests/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,5 @@ require (
golang.org/x/tools v0.0.0-20190102183724-79186431cf29 // indirect
honnef.co/go/tools v0.0.0-20190102075043-fe93b0e3b36b // indirect
)

go 1.13
14 changes: 0 additions & 14 deletions tests/pkg/clean/build_tools_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -145,20 +145,6 @@ func TestErrCheck(t *testing.T) {
a.NoError(err) // Should have no complaints in clean package
}

// Test staticcheck.
func TestStaticcheck(t *testing.T) {
a := assert.New(t)

// Test "make staticcheck"
v, err := exec.Command("bash", "-c", "make staticcheck").Output()
a.Error(err) // Should have one complaint about bad_staticcheck_code.go
a.Contains(string(v), "pkg/dirtyComplex/bad_staticcheck_code.go")

// Test "make SRC_DIRS=pkg/clean staticcheck" to limit to just clean directories
v, err = exec.Command("bash", "-c", "make staticcheck SRC_DIRS=pkg/clean").Output()
a.NoError(err, "error output from staticcheck: %v", string(v)) // Should have no complaints in clean package
}

// Test misspell.
func TestMisspell(t *testing.T) {
a := assert.New(t)
Expand Down
2 changes: 2 additions & 0 deletions tests/standard_target/go.mod
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
module github.com/drud/build-tools/tests/standard_target/cmd

go 1.13

0 comments on commit c0092fa

Please sign in to comment.