go/types: remove spurious quotes around package name in diagnostic message#71539
go/types: remove spurious quotes around package name in diagnostic message#71539eikemeier wants to merge 1 commit intogolang:masterfrom
Conversation
|
This PR (HEAD: c2538a4) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/646135. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Robert Findley: Patch Set 1: Code-Review+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Robert Findley: Patch Set 1: Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Go LUCI: Patch Set 1: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-02-03T14:50:24Z","revision":"6f04ef3d19ee927677d1a1e5232653c52ffbe018"} Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Robert Findley: Patch Set 1: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Go LUCI: Patch Set 1: This CL has failed the run. Reason: Tryjob golang/try/gotip-linux-386 has failed with summary (view all results):
Build or test failure, click here for results. To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Go LUCI: Patch Set 1: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Robert Findley: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Robert Griesemer: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Ian Lance Taylor: Patch Set 1: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
This PR (HEAD: f37068f) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/646135. Important tips:
|
|
Message from Robert Findley: Patch Set 2: Code-Review+1 Commit-Queue+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Go LUCI: Patch Set 2: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-02-24T18:19:20Z","revision":"53475aef4bc782e49d7310248d31e6997d406492"} Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Robert Findley: Patch Set 2: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Go LUCI: Patch Set 2: This CL has failed the run. Reason: Tryjob golang/try/gotip-linux-amd64 has failed with summary (view all results):
Build or test failure, click here for results. To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
|
Message from Go LUCI: Patch Set 2: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/646135. |
Signed-off-by: Oliver Eikemeier <eikemeier@fillmore-labs.com>
When you have a type error involving a type from a package whose name is used by multiple packages, spurious quotes are added around the package name.
Example:
does not build with the message
it is expected to not build, but the quotes around “"golang.org/x/oauth2"” are unexpected.