[vscode-go] goInstallTools: show PATH entries when go is missing

6 views
Skip to first unread message

Gerrit Bot (Gerrit)

unread,
May 2, 2026, 10:43:19 PM (9 days ago) May 2
to goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com

Gerrit Bot has uploaded the change for review

Commit message

goInstallTools: show PATH entries when go is missing

Problem

When the extension cannot find the go binary, the existing prompt points
users to install Go but does not help diagnose PATH mismatches inside VS
Code. Instead, it shows a very disruptive blog of paths that is usually cutoff
as it does not fit the notification UI.

Change

This change updates the missing-go prompt to offer a "Show PATH Entries"
action alongside the download link.

The PATH action opens a quick pick with the PATH entries searched for the
go binary. This makes it easier to spot missing or unexpected
directories.

The prompt message is also trimmed to report GOROOT separately and avoid
dumping the full PATH into the error text.

Implementation

The choice handling is split into small local helpers in
goInstallTools.ts. Brief comments are also added for the new prompt
actions and the PATH parsing helper.

Testing

Tested with npm run bundle-dev
Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
GitHub-Last-Rev: 44712bf71326390caf6610a30a2c1c54055c6b0d
GitHub-Pull-Request: golang/vscode-go#4028

Change diff

diff --git a/extension/src/goInstallTools.ts b/extension/src/goInstallTools.ts
index 97cd781..9ea8365 100644
--- a/extension/src/goInstallTools.ts
+++ b/extension/src/goInstallTools.ts
@@ -720,21 +720,63 @@

let suggestedDownloadGo = false;

-async function suggestDownloadGo() {
- const msg =
- `Failed to find the "go" binary in either GOROOT(${getCurrentGoRoot()}) or PATH(${getEnvPath()}). ` +
- 'Check PATH, or Install Go and reload the window. ' +
- "If PATH isn't what you expected, see https://github.com/golang/vscode-go/issues/971";
+// Actions shown when the extension cannot locate the go binary.
+const SHOW_PATH_ENTRIES_ACTION = 'Show PATH Entries';
+const GO_DOWNLOAD_PAGE_ACTION = 'Go to Download Page';
+const GO_DOWNLOAD_URL = 'https://golang.org/dl/';

- if (suggestedDownloadGo) {
- vscode.window.showErrorMessage(msg);
+// Split PATH using the platform delimiter and drop empty entries for display.
+function getPathEntries(): string[] {
+ return (getEnvPath() ?? '')
+ .split(path.delimiter)
+ .map((entry: string) => entry.trim())
+ .filter(Boolean);
+}
+
+async function handleDownloadGoChoice(choice: string | undefined): Promise<void> {
+ if (choice === SHOW_PATH_ENTRIES_ACTION) {
+ await showPathEntriesQuickPick();
return;
}

- const choice = await vscode.window.showErrorMessage(msg, 'Go to Download Page');
- if (choice === 'Go to Download Page') {
- vscode.env.openExternal(vscode.Uri.parse('https://golang.org/dl/'));
+ if (choice === GO_DOWNLOAD_PAGE_ACTION) {
+ await vscode.env.openExternal(vscode.Uri.parse(GO_DOWNLOAD_URL));
}
+ return;
+}
+
+async function showPathEntriesQuickPick() {
+ const pathEntries = getPathEntries();
+
+ if (!pathEntries.length) {
+ await vscode.window.showInformationMessage('PATH is empty.');
+ return;
+ }
+
+ await vscode.window.showQuickPick(pathEntries, {
+ title: 'PATH Entries Searched for go',
+ placeHolder: 'Entries from PATH used when searching for the go binary',
+ ignoreFocusOut: true
+ });
+}
+
+async function suggestDownloadGo() {
+ const goRoot = getCurrentGoRoot() ?? '(not set)';
+ const message =
+ `Failed to find the "go" binary in either GOROOT(${goRoot}) or PATH. ` +
+ 'Check PATH, or install Go and reload the window.';
+ const choice = await vscode.window.showErrorMessage(
+ message,
+ SHOW_PATH_ENTRIES_ACTION,
+ GO_DOWNLOAD_PAGE_ACTION
+ );
+
+ if (suggestedDownloadGo) {
+ await handleDownloadGoChoice(choice);
+ return;
+ }
+
+ await handleDownloadGoChoice(choice);
suggestedDownloadGo = true;
}

Change information

Files:
  • M extension/src/goInstallTools.ts
Change size: M
Delta: 1 file changed, 52 insertions(+), 10 deletions(-)
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement is not satisfiedCode-Review
  • requirement satisfiedNo-Unresolved-Comments
  • requirement is not satisfiedReview-Enforcement
  • requirement is not satisfiedTryBots-Pass
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: newchange
Gerrit-Project: vscode-go
Gerrit-Branch: master
Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
Gerrit-Change-Number: 773540
Gerrit-PatchSet: 1
Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
unsatisfied_requirement
satisfied_requirement
open
diffy

Gopher Robot (Gerrit)

unread,
May 2, 2026, 10:43:22 PM (9 days ago) May 2
to Gerrit Bot, goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com

Gopher Robot added 1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Gopher Robot . unresolved

I spotted some possible problems with your PR:

  1. You usually need to reference a bug number for all but trivial or cosmetic fixes. For the vscode-go repo, the format is usually 'Fixes golang/vscode-go#1234' or 'Updates golang/vscode-go#1234' at the end of the commit message. Should you have a bug reference?

Please address any problems by updating the GitHub PR.

When complete, mark this comment as 'Done' and click the [blue 'Reply' button](https://go.dev/wiki/GerritBot#i-left-a-reply-to-a-comment-in-gerrit-but-no-one-but-me-can-see-it) above. These findings are based on heuristics; if a finding does not apply, briefly reply here saying so.

To update the commit title or commit message body shown here in Gerrit, you must edit the GitHub PR title and PR description (the first comment) in the GitHub web interface using the 'Edit' button or 'Edit' menu entry there. Note: pushing a new commit to the PR will not automatically update the commit message used by Gerrit.

For more details, see:

(In general for Gerrit code reviews, the change author is expected to [log in to Gerrit](https://go-review.googlesource.com/login/) with a Gmail or other Google account and then close out each piece of feedback by marking it as 'Done' if implemented as suggested or otherwise reply to each review comment. See the [Review](https://go.dev/doc/contribute#review) section of the Contributing Guide for details.)

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement is not satisfiedReview-Enforcement
    • requirement is not satisfiedTryBots-Pass
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: vscode-go
    Gerrit-Branch: master
    Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
    Gerrit-Change-Number: 773540
    Gerrit-PatchSet: 1
    Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
    Gerrit-CC: Gopher Robot <go...@golang.org>
    Gerrit-Comment-Date: Sun, 03 May 2026 02:43:17 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    unsatisfied_requirement
    open
    diffy

    Gopher Robot (Gerrit)

    unread,
    May 2, 2026, 10:48:31 PM (9 days ago) May 2
    to Gerrit Bot, goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com

    Message from Gopher Robot

    Congratulations on opening your first change. Thank you for your contribution!

    Next steps:
    A maintainer will review your change and provide feedback. See
    https://go.dev/doc/contribute#review for more info and tips to get your
    patch through code review.

    Most changes in the Go project go through a few rounds of revision. This can be
    surprising to people new to the project. The careful, iterative review process
    is our way of helping mentor contributors and ensuring that their contributions
    have a lasting impact.

    Open in Gerrit

    Related details

    Attention set is empty
    Submit Requirements:
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement is not satisfiedReview-Enforcement
    • requirement is not satisfiedTryBots-Pass
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: vscode-go
    Gerrit-Branch: master
    Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
    Gerrit-Change-Number: 773540
    Gerrit-PatchSet: 1
    Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
    Gerrit-CC: Gopher Robot <go...@golang.org>
    Gerrit-Comment-Date: Sun, 03 May 2026 02:48:26 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: No
    unsatisfied_requirement
    open
    diffy

    Madeline Kalil (Gerrit)

    unread,
    May 4, 2026, 10:22:38 AM (8 days ago) May 4
    to Tarik C. Brown, Gerrit Bot, goph...@pubsubhelper.golang.org, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
    Attention needed from Hongxiang Jiang

    Madeline Kalil voted Commit-Queue+1

    Commit-Queue+1
    Open in Gerrit

    Related details

    Attention is currently required from:
    • Hongxiang Jiang
    Submit Requirements:
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement is not satisfiedReview-Enforcement
    • requirement is not satisfiedTryBots-Pass
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: vscode-go
    Gerrit-Branch: master
    Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
    Gerrit-Change-Number: 773540
    Gerrit-PatchSet: 1
    Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
    Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
    Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
    Gerrit-CC: Gopher Robot <go...@golang.org>
    Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
    Gerrit-Comment-Date: Mon, 04 May 2026 14:22:34 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    unsatisfied_requirement
    open
    diffy

    Tarik C. Brown (Gerrit)

    unread,
    May 4, 2026, 1:44:43 PM (7 days ago) May 4
    to Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Madeline Kalil, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
    Attention needed from Hongxiang Jiang and Madeline Kalil

    Tarik C. Brown added 1 comment

    Patchset-level comments
    Gopher Robot . unresolved

    I spotted some possible problems with your PR:

      1. You usually need to reference a bug number for all but trivial or cosmetic fixes. For the vscode-go repo, the format is usually 'Fixes golang/vscode-go#1234' or 'Updates golang/vscode-go#1234' at the end of the commit message. Should you have a bug reference?

    Please address any problems by updating the GitHub PR.

    When complete, mark this comment as 'Done' and click the [blue 'Reply' button](https://go.dev/wiki/GerritBot#i-left-a-reply-to-a-comment-in-gerrit-but-no-one-but-me-can-see-it) above. These findings are based on heuristics; if a finding does not apply, briefly reply here saying so.

    To update the commit title or commit message body shown here in Gerrit, you must edit the GitHub PR title and PR description (the first comment) in the GitHub web interface using the 'Edit' button or 'Edit' menu entry there. Note: pushing a new commit to the PR will not automatically update the commit message used by Gerrit.

    For more details, see:

    (In general for Gerrit code reviews, the change author is expected to [log in to Gerrit](https://go-review.googlesource.com/login/) with a Gmail or other Google account and then close out each piece of feedback by marking it as 'Done' if implemented as suggested or otherwise reply to each review comment. See the [Review](https://go.dev/doc/contribute#review) section of the Contributing Guide for details.)

    Tarik C. Brown

    There should not be a bug reference, this is a mostly cosmetic change with a slight redesign of existing behavior. There is no fundamental change to what this feature originally intended. It’s just been made a bit more user friendly.

    It also seems that the checks in my PR are breaking due to unrelated changes:

            2026/05/04 14:27:16 unknown type "int64"

    This build error does not reproduce locally and seems caused by the introduction of Go 1.26

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Hongxiang Jiang
    • Madeline Kalil
    Submit Requirements:
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement is not satisfiedReview-Enforcement
    • requirement is not satisfiedTryBots-Pass
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: vscode-go
    Gerrit-Branch: master
    Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
    Gerrit-Change-Number: 773540
    Gerrit-PatchSet: 1
    Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
    Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
    Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
    Gerrit-CC: Gopher Robot <go...@golang.org>
    Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
    Gerrit-Attention: Madeline Kalil <mka...@google.com>
    Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
    Gerrit-Comment-Date: Mon, 04 May 2026 17:44:39 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    Comment-In-Reply-To: Gopher Robot <go...@golang.org>
    unsatisfied_requirement
    open
    diffy

    Tarik C. Brown (Gerrit)

    unread,
    May 4, 2026, 1:53:06 PM (7 days ago) May 4
    to Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Madeline Kalil, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
    Attention needed from Hongxiang Jiang and Madeline Kalil

    Tarik C. Brown added 1 comment

    Patchset-level comments
    Gopher Robot . resolved

    I spotted some possible problems with your PR:

      1. You usually need to reference a bug number for all but trivial or cosmetic fixes. For the vscode-go repo, the format is usually 'Fixes golang/vscode-go#1234' or 'Updates golang/vscode-go#1234' at the end of the commit message. Should you have a bug reference?

    Please address any problems by updating the GitHub PR.

    When complete, mark this comment as 'Done' and click the [blue 'Reply' button](https://go.dev/wiki/GerritBot#i-left-a-reply-to-a-comment-in-gerrit-but-no-one-but-me-can-see-it) above. These findings are based on heuristics; if a finding does not apply, briefly reply here saying so.

    To update the commit title or commit message body shown here in Gerrit, you must edit the GitHub PR title and PR description (the first comment) in the GitHub web interface using the 'Edit' button or 'Edit' menu entry there. Note: pushing a new commit to the PR will not automatically update the commit message used by Gerrit.

    For more details, see:

    (In general for Gerrit code reviews, the change author is expected to [log in to Gerrit](https://go-review.googlesource.com/login/) with a Gmail or other Google account and then close out each piece of feedback by marking it as 'Done' if implemented as suggested or otherwise reply to each review comment. See the [Review](https://go.dev/doc/contribute#review) section of the Contributing Guide for details.)

    Tarik C. Brown

    There should not be a bug reference, this is a mostly cosmetic change with a slight redesign of existing behavior. There is no fundamental change to what this feature originally intended. It’s just been made a bit more user friendly.

    It also seems that the checks in my PR are breaking due to unrelated changes:

            2026/05/04 14:27:16 unknown type "int64"

    This build error does not reproduce locally and seems caused by the introduction of Go 1.26

    Tarik C. Brown

    Done

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Hongxiang Jiang
    • Madeline Kalil
    Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 1
      Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
      Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
      Gerrit-CC: Gopher Robot <go...@golang.org>
      Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Madeline Kalil <mka...@google.com>
      Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Comment-Date: Mon, 04 May 2026 17:53:01 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: No
      Comment-In-Reply-To: Tarik C. Brown <tarikcb...@gmail.com>
      Comment-In-Reply-To: Gopher Robot <go...@golang.org>
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy

      Madeline Kalil (Gerrit)

      unread,
      May 8, 2026, 10:50:49 AM (4 days ago) May 8
      to Tarik C. Brown, Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
      Attention needed from Hongxiang Jiang

      Madeline Kalil voted Commit-Queue+1

      Commit-Queue+1
      Open in Gerrit

      Related details

      Attention is currently required from:
      • Hongxiang Jiang
      Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 1
      Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
      Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
      Gerrit-CC: Gopher Robot <go...@golang.org>
      Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Comment-Date: Fri, 08 May 2026 14:50:46 +0000
      Gerrit-HasComments: No
      Gerrit-Has-Labels: Yes
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy

      Madeline Kalil (Gerrit)

      unread,
      May 8, 2026, 10:59:34 AM (3 days ago) May 8
      to Tarik C. Brown, Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
      Attention needed from Hongxiang Jiang

      Madeline Kalil added 1 comment

      Patchset-level comments
      Madeline Kalil . resolved

      I believe you'll need to rebase this CL on the master branch in order for the tests to pass

      Open in Gerrit

      Related details

      Attention is currently required from:
      • Hongxiang Jiang
      Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 1
      Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
      Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
      Gerrit-CC: Gopher Robot <go...@golang.org>
      Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Comment-Date: Fri, 08 May 2026 14:59:30 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: No
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy

      Tarik C. Brown (Gerrit)

      unread,
      May 8, 2026, 5:44:02 PM (3 days ago) May 8
      to Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Madeline Kalil, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
      Attention needed from Hongxiang Jiang and Madeline Kalil

      Tarik C. Brown added 1 comment

      Patchset-level comments
      Madeline Kalil . resolved

      I believe you'll need to rebase this CL on the master branch in order for the tests to pass

      Tarik C. Brown

      Thanks, the PR should be rebased and ready for the pipeline now.

      Open in Gerrit

      Related details

      Attention is currently required from:
      • Hongxiang Jiang
      • Madeline Kalil
      Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 1
      Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
      Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
      Gerrit-CC: Gopher Robot <go...@golang.org>
      Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Madeline Kalil <mka...@google.com>
      Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Comment-Date: Fri, 08 May 2026 21:43:57 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: No
      Comment-In-Reply-To: Madeline Kalil <mka...@google.com>
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy

      Gerrit Bot (Gerrit)

      unread,
      May 8, 2026, 5:49:26 PM (3 days ago) May 8
      to Tarik C. Brown, goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com
      Attention needed from Hongxiang Jiang and Madeline Kalil

      Gerrit Bot uploaded new patchset

      Gerrit Bot uploaded patch set #2 to this change.
      Following approvals got outdated and were removed:
      Open in Gerrit

      Related details

      Attention is currently required from:
      • Hongxiang Jiang
      • Madeline Kalil
      Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: newpatchset
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 2
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy

      Tarik C. Brown (Gerrit)

      unread,
      5:03 PM (6 hours ago) 5:03 PM
      to Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Madeline Kalil, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
      Attention needed from Hongxiang Jiang and Madeline Kalil

      Tarik C. Brown added 1 comment

      Patchset-level comments
      Madeline Kalil . resolved

      I believe you'll need to rebase this CL on the master branch in order for the tests to pass

      Tarik C. Brown

      Thanks, the PR should be rebased and ready for the pipeline now.

      Tarik C. Brown

      @mka...@google.com Can you run the CL again?

      Open in Gerrit

      Related details

      Attention is currently required from:
      • Hongxiang Jiang
      • Madeline Kalil
      Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 2
      Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
      Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
      Gerrit-CC: Gopher Robot <go...@golang.org>
      Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Madeline Kalil <mka...@google.com>
      Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Comment-Date: Mon, 11 May 2026 21:03:52 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: No
      Comment-In-Reply-To: Madeline Kalil <mka...@google.com>
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy

      Madeline Kalil (Gerrit)

      unread,
      5:06 PM (6 hours ago) 5:06 PM
      to Tarik C. Brown, Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
      Attention needed from Hongxiang Jiang

      Madeline Kalil voted Commit-Queue+1

      Commit-Queue+1
      Open in Gerrit

      Related details

      Attention is currently required from:
      • Hongxiang Jiang
      Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 2
      Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
      Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
      Gerrit-CC: Gopher Robot <go...@golang.org>
      Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Comment-Date: Mon, 11 May 2026 21:06:04 +0000
      Gerrit-HasComments: No
      Gerrit-Has-Labels: Yes
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy

      Madeline Kalil (Gerrit)

      unread,
      5:06 PM (6 hours ago) 5:06 PM
      to Tarik C. Brown, Gerrit Bot, goph...@pubsubhelper.golang.org, golang...@luci-project-accounts.iam.gserviceaccount.com, Hongxiang Jiang, Gopher Robot, golang-co...@googlegroups.com
      Attention needed from Hongxiang Jiang and Tarik C. Brown

      Madeline Kalil added 1 comment

      Patchset-level comments
      Madeline Kalil . resolved

      I believe you'll need to rebase this CL on the master branch in order for the tests to pass

      Tarik C. Brown

      Thanks, the PR should be rebased and ready for the pipeline now.

      Tarik C. Brown

      @mka...@google.com Can you run the CL again?

      Madeline Kalil

      Ran it again, although I have not had time to review this change yet.

      Open in Gerrit

      Related details

      Attention is currently required from:
      • Hongxiang Jiang
      • Tarik C. Brown
      Submit Requirements:
      • requirement is not satisfiedCode-Review
      • requirement satisfiedNo-Unresolved-Comments
      • requirement is not satisfiedReview-Enforcement
      • requirement is not satisfiedTryBots-Pass
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: vscode-go
      Gerrit-Branch: master
      Gerrit-Change-Id: I8f81230e22f8ed8ff295a37fce877e280a445c3f
      Gerrit-Change-Number: 773540
      Gerrit-PatchSet: 2
      Gerrit-Owner: Gerrit Bot <letsus...@gmail.com>
      Gerrit-Reviewer: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Reviewer: Madeline Kalil <mka...@google.com>
      Gerrit-CC: Gopher Robot <go...@golang.org>
      Gerrit-CC: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Tarik C. Brown <tarikcb...@gmail.com>
      Gerrit-Attention: Hongxiang Jiang <hxj...@golang.org>
      Gerrit-Comment-Date: Mon, 11 May 2026 21:06:24 +0000
      unsatisfied_requirement
      satisfied_requirement
      open
      diffy
      Reply all
      Reply to author
      Forward
      0 new messages