Skip to content

Commit 41a13ec

Browse files
committed
Fix comment in analyse Action
1 parent 9a5a628 commit 41a13ec

File tree

3 files changed

+9
-3
lines changed

3 files changed

+9
-3
lines changed

lib/analyze-action.js

Lines changed: 4 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/analyze-action.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/analyze-action.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,10 @@ async function run() {
232232
logger,
233233
);
234234

235-
// Check that the Go wrapper script still exists, if set
235+
// Check that `which go` still points at the wrapper script we installed in the `init` Action,
236+
// if the corresponding environment variable is set. This is to ensure that there isn't a step
237+
// in the workflow after the `init` step which installs a different version of Go and takes
238+
// precedence in the PATH, thus potentially circumventing our workaround that allows tracing to work.
236239
const goWrapperPath = process.env[EnvVar.GO_BINARY_LOCATION];
237240

238241
if (goWrapperPath !== undefined) {

0 commit comments

Comments
 (0)