hack/tests/sanity-check.sh: ignore go.mod changes caused by go vet #2073
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the change:
This PR updates the sanity check script to ignore changes to go.mod caused by
go vet
(and other go commands).Motivation for the change:
Many
go
commands change thego.mod
file if it does not match what the go tooling expects.go.mod
files managed by go1.12 are changed when running go commands with go1.13. This causes problems with the sanity check script when trying to migrate from go1.12 to go1.13.During the transition, we can ignore
go.mod
file changes that occur whengo vet
runs earlier in the sanity check script. After the transition, we will revert this PR so that futurego.mod
files are checked for adherance to the go tooling expectations.Order of operations to complete go1.13 migration