-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
x/tools/gopls: "could not compute pos to range for %v: %v" bug (reported by telemetry) #65960
Labels
gopls/telemetry-wins
gopls
Issues related to the Go language server, gopls.
NeedsInvestigation
Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Tools
This label describes issues relating to any tools in the x/tools repository.
Milestone
Comments
adonovan
added
NeedsInvestigation
Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
gopls
Issues related to the Go language server, gopls.
Tools
This label describes issues relating to any tools in the x/tools repository.
gopls/telemetry-wins
labels
Feb 27, 2024
Change https://go.dev/cl/567416 mentions this issue: |
gopherbot
pushed a commit
to golang/tools
that referenced
this issue
Feb 28, 2024
Updates golang/go#65960 Change-Id: I01a416a0cf9cf8e13195c0d9405008ded1a9c53a Reviewed-on: https://go-review.googlesource.com/c/tools/+/567416 Auto-Submit: Alan Donovan <adonovan@google.com> Reviewed-by: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
This stack
Issue created by golang.org/x/tools/gopls/internal/telemetry/cmd/stacks. |
A new variant: // debugging #65960
if _, err := safetoken.Offset(pgf.Tok, end); err != nil {
bug.Reportf("ReadGo116ErrorData returned invalid end: %v", err)
} |
Change https://go.dev/cl/621876 mentions this issue: |
gopherbot
pushed a commit
to golang/tools
that referenced
this issue
Oct 22, 2024
Refine a few bug reports related to incorrectly positioned type checker errors, by differentiating the 'fixed file' case from the case with no fixed files. Since we haven't made progress on these bugs, we need more information to guide our debugging. If the bugs only occur in the presence of AST fixes, we can probably downgrade them to not be a bug: if we're fixing the AST, we won't display type checker errors anyway. Nevertheless, leave the bug reports for now, so that we can collect data. For golang/go#65960 For golang/go#66765 For golang/go#66766 Change-Id: I2060c897d249cdd5cc3e7bb183d3f563987bec57 Reviewed-on: https://go-review.googlesource.com/c/tools/+/621876 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Alan Donovan <adonovan@google.com> Auto-Submit: Robert Findley <rfindley@google.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
gopls/telemetry-wins
gopls
Issues related to the Go language server, gopls.
NeedsInvestigation
Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Tools
This label describes issues relating to any tools in the x/tools repository.
This stack
q8FyvQ
was reported by telemetry:Note variants:
bug.Reportf("TypeErrorEndPos returned invalid end: %v", err)
bug.Reportf("ReadGo116ErrorData returned invalid end: %v", err)
Issue created by golang.org/x/tools/gopls/internal/telemetry/cmd/stacks.
Dups: rnoW6w 7zL2qQ pT-4Ug
The text was updated successfully, but these errors were encountered: