-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[tasks] stop the build if GO111MODULE is set to on. #4995
Conversation
ac2e968
to
c266667
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM overal!
Let's add this check to inv deps
as well? + I made a few inline nits
Co-Authored-By: Olivier Vielpeau <olivielpeau@users.noreply.github.com>
…-agent into remeh/invoke-go111module
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small nit but apart from that LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 on Albert's comment + a few nits
Co-Authored-By: Olivier Vielpeau <olivielpeau@users.noreply.github.com>
Co-Authored-By: Olivier Vielpeau <olivielpeau@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
What does this PR do?
Add a step in all builds task to stop the build if
GO111MODULE
is set toon
.Motivation
The datadog-agent is not using the Go modules yet. The default behavior of the Go modules while compiling the Agent in a GOPATH is fine (it's not using the Go modules because we're in a GOPATH and there is no go.mod file available), however, if GO111MODULE has been forced to on in the user environment, the build would fail.