Skip to content

Commit

Permalink
Merge pull request #1926 from github/mbg/suppress-go-workaround-warning
Browse files Browse the repository at this point in the history
Suppress Go workaround warning if autobuilder ran
  • Loading branch information
mbg authored Oct 9, 2023
2 parents 0d5c2e0 + ffaea6c commit faf91ca
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
3 changes: 2 additions & 1 deletion lib/analyze-action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/analyze-action.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 4 additions & 1 deletion src/analyze-action.ts
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,10 @@ async function run() {
// precedence in the PATH, thus potentially circumventing our workaround that allows tracing to work.
const goWrapperPath = process.env[EnvVar.GO_BINARY_LOCATION];

if (goWrapperPath !== undefined) {
if (
process.env[EnvVar.DID_AUTOBUILD_GOLANG] !== "true" &&
goWrapperPath !== undefined
) {
const goBinaryPath = await safeWhich("go");

if (goWrapperPath !== goBinaryPath) {
Expand Down

0 comments on commit faf91ca

Please sign in to comment.