Skip to content

Commit

Permalink
Explicitly run go fmt on csi/node_test.go
Browse files Browse the repository at this point in the history
go 1.11's "go fmt" has a backwards incompatible formatting
change due to golang/go#26228
which only affects the csi/csi_test.go file.

This is a temporary workaround for difference in go fmt between
1.9/1.10 and 1.11+, to fix travis.  We can remove this when we drop
go 1.9 and go 1.10 from travis.

Signed-off-by: Craig Rodrigues <craig@portworx.com>
  • Loading branch information
Craig Rodrigues committed Apr 15, 2019
1 parent a38d6e9 commit a4da2f9
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@ install:
- go get -u github.com/go-swagger/go-swagger/cmd/swagger
script:
- git-validation -run DCO,short-subject
# Temporary hack to work around 'go fmt' difference between go 1.9/1.10 and 1.11+, only affects node_test.go
- go fmt csi/node_test.go
- go fmt csi/v0.3/node_test.go
- go fmt $(go list ./... | grep -v vendor) | wc -l | grep 0
- make docker-proto
- git diff $(find . -name "*.pb.*go" -o -name "api.swagger.json" | grep -v vendor) | wc -l | grep "^0"
Expand Down

0 comments on commit a4da2f9

Please sign in to comment.