qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-10-01 13:28:37 -06:00
Code Releases Activity
f66ef90017
go/internal/lsp/testdata/summary.txt.golden

26 lines
543 B
Plaintext
Raw Normal View History

use a golden file for the expected test counts This makes it much easier to keep them up to date. It is also less fragile against accidental changes. Change-Id: If119f8527c0896d210650859960e77f3e0fa5a99 Reviewed-on: https://go-review.googlesource.com/c/tools/+/197505 Run-TryBot: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-09-26 10:54:25 -06:00
-- summary --
internal/lsp/source: improve completion after accidental keywords Completion often fails when the completion prefix happens to be a keyword. We previously tried to fix this with AST surgery, but often the accidental keyword is not apparent looking at the AST. For example: chan<> foo() parses as CallExpr{Fun: ChanType{Value: Ident{"foo"}}} with very few hints that something is wrong, and: default foo() is completely omitted from the AST. Rather than look in the AST, we now instead manually look for a keyword token that contains the completion position. If we find one, we treat that as our surrounding identifier. Updates golang/go#34332. Change-Id: I68ed0dd905848c0eae61f39ecb8b73adb1e72746 Reviewed-on: https://go-review.googlesource.com/c/tools/+/216961 Run-TryBot: Muir Manders <muir@mnd.rs> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2020-01-29 22:52:22 -07:00
CompletionsCount = 226
internal/lsp/source: support dereferencing for completion In cases like: var j *int var i int = <> We will now provide "*j" as a completion candidate. Change-Id: I1d35c2dca4864f13f7534e15b17450d784985557 Reviewed-on: https://go-review.googlesource.com/c/tools/+/215358 Run-TryBot: Muir Manders <muir@mnd.rs> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2020-01-18 14:33:26 -07:00
CompletionSnippetCount = 66
internal/imports: load test exports of package under test x_tests can access exports from the test variant of the package under test. Change loadExportsFromFiles to understand that mode, and use it where appropriate. I didn't want to come up with a cache key for for the test variant, so for now we bypass the cache in these situations. Fixes golang/go#29979. Change-Id: I9959a08da97bbee64c5bcd56e06f548486693156 Reviewed-on: https://go-review.googlesource.com/c/tools/+/213221 Run-TryBot: Heschi Kreinick <heschi@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2020-01-03 16:01:15 -07:00
UnimportedCompletionsCount = 9
use a golden file for the expected test counts This makes it much easier to keep them up to date. It is also less fragile against accidental changes. Change-Id: If119f8527c0896d210650859960e77f3e0fa5a99 Reviewed-on: https://go-review.googlesource.com/c/tools/+/197505 Run-TryBot: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-09-26 10:54:25 -06:00
DeepCompletionsCount = 5
internal/lsp: support taking address for completion candidates We now support taking the address of objects to make better completion candidates. For example: i := 123 var p *int = <> // now you get a candidate for "&i" This required that we track addressability better, particularly when searching for deep candidates. Now each candidate knows if it is addressable, and the deep search propagates addressability to child candidates appropriately. The basic propagation logic is: - In-scope *types.Var candidates are addressable. This handles your basic "foo" variable whose address if "&foo". - Surrounding selector is addressable based on type checker info. This knows "foo.bar.<>" is addressable but "foo.bar().<>" isn't - When evaluating deep completions, fields after a function call lose addressability, but fields after a pointer regain addressability. For example, "foo.bar()" isn't addressable, but "foo.bar().baz" is addressable if "bar()" returns a pointer. Fixes golang/go#36132. Change-Id: I6a8659eb8c203262aedf86844ac39a2d1e81ecc4 Reviewed-on: https://go-review.googlesource.com/c/tools/+/212399 Run-TryBot: Muir Manders <muir@mnd.rs> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-12-22 10:58:14 -07:00
FuzzyCompletionsCount = 8
internal/lsp/source: improve completion involving multiple return values For example: // Prefer functions that return one or two values. Previously // we had no preference. foo, bar := <> // Prefer functions that return "(int)" or "(int, ??)". Previously we // only preferred the former. var foo int foo, bar := <> // Prefer functions that return "(int)" or "(int, int)". Previously we // only preferred the former. var foo func(int, int) foo(<>) In the above example, we don't handle "foo" being variadic yet. I also took the liberty to break up matchingCandidate() into separate functions since it was getting rather long. Updates golang/go#36540. Change-Id: I9140dd989dfde1ddcfcd9d2a14198045c02587f2 Reviewed-on: https://go-review.googlesource.com/c/tools/+/215537 Run-TryBot: Muir Manders <muir@mnd.rs> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2020-01-20 15:50:36 -07:00
RankedCompletionsCount = 83
use a golden file for the expected test counts This makes it much easier to keep them up to date. It is also less fragile against accidental changes. Change-Id: If119f8527c0896d210650859960e77f3e0fa5a99 Reviewed-on: https://go-review.googlesource.com/c/tools/+/197505 Run-TryBot: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-09-26 10:54:25 -06:00
CaseSensitiveCompletionsCount = 4
internal/span, internal/lsp: fix URI escaping We had previously worked around a VS Code URI bug by unescaping URIs. This is incorrect, so stop doing it and then add a specific workaround just for that one bug. Fixes golang/go#36999 Change-Id: I92f1a5f71749af7a6b1020eee1272586515f7084 Reviewed-on: https://go-review.googlesource.com/c/tools/+/217599 Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com>
2020-02-03 16:07:33 -07:00
DiagnosticsCount = 38
use a golden file for the expected test counts This makes it much easier to keep them up to date. It is also less fragile against accidental changes. Change-Id: If119f8527c0896d210650859960e77f3e0fa5a99 Reviewed-on: https://go-review.googlesource.com/c/tools/+/197505 Run-TryBot: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-09-26 10:54:25 -06:00
FoldingRangesCount = 2
FormatCount = 6
internal/lsp/source: start import diffs at byte 0 I assumed that f.Pos() would be the first byte of the file, but it's the position of the package declaration. This kills the file. Just use 0. Fixes golang/go#35458. Change-Id: Ic77c93344c71435ef8e5624c2f2defb619139a15 Reviewed-on: https://go-review.googlesource.com/c/tools/+/206145 Run-TryBot: Heschi Kreinick <heschi@google.com> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-11-08 11:43:54 -07:00
ImportCount = 7
use a golden file for the expected test counts This makes it much easier to keep them up to date. It is also less fragile against accidental changes. Change-Id: If119f8527c0896d210650859960e77f3e0fa5a99 Reviewed-on: https://go-review.googlesource.com/c/tools/+/197505 Run-TryBot: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-09-26 10:54:25 -06:00
SuggestedFixCount = 1
internal/lsp: don't show links in hover for test functions source.Identifier previously was used for references and rename, so it needed to take a package policy. Now, it's only used for definition and hover, so it should always be the narrowest package handle. We can use this fact to determine if the identifier is located in its declaring package, and if that package is a test variant, we don't link to the documentation on pkg.go.dev, since it doesn't exist. Change-Id: I5686828858a3feafb8ff2e4c5964b562f66db9fa Reviewed-on: https://go-review.googlesource.com/c/tools/+/217137 Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com>
2020-01-30 17:24:33 -07:00
DefinitionsCount = 45
use a golden file for the expected test counts This makes it much easier to keep them up to date. It is also less fragile against accidental changes. Change-Id: If119f8527c0896d210650859960e77f3e0fa5a99 Reviewed-on: https://go-review.googlesource.com/c/tools/+/197505 Run-TryBot: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-09-26 10:54:25 -06:00
TypeDefinitionsCount = 2
internal/lsp: add highlighting for import statement This change adds highlights for imports when the cursor is over the use of that import. It also adds it for the opposite direction when the cursor is on the import, it will highlight uses of that import. Fixes golang/go#36590 Change-Id: Ifd04d81ec9b4fdf2be1b763f31b44d0ef7d92f47 Reviewed-on: https://go-review.googlesource.com/c/tools/+/215258 Reviewed-by: Rebecca Stambler <rstambler@golang.org> Run-TryBot: Rohan Challa <rohan@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
2020-01-17 08:54:55 -07:00
HighlightsCount = 52
internal/lsp: refactor find-references and rename The main goal is to push the package variant logic from internal/lsp into internal/lsp/source so all users of internal/lsp/source benefit. "references" and "rename" now have top-level source.References() and source.Rename() entry points (as opposed to hanging off source.Identifier()). I expanded objectsAtProtocolPos() to know about implicit objects (type switch and import spec), and to handle *ast.ImportSpec generically. This gets rid of special case handling of *types.PkgName in various places. The biggest practical benefit, though, is that "references" no longer needs to compute the objectpath for every types.Object comparison it does, instead using direct types.Object equality. This speeds up "references" and "rename" a lot. Two other notable improvements that fell out of not using source.Identifier()'s logic: - Finding references on an embedded field now shows references to the field, not the type being embedded. - Finding references on an imported object now works correctly (previously it searched the importing package's dependents rather than the imported package's dependents). Finally, I refactored findIdentifier() to use pathEnclosingObjNode() instead of astutil.PathEnclosingInterval. Now we only need a single call to get the path because pathEnclosingObjNode() has the "try pos || try pos-1" logic built in. Change-Id: I667be9bed6ad83912404b90257c5c1485b3a7025 Reviewed-on: https://go-review.googlesource.com/c/tools/+/211999 Run-TryBot: Muir Manders <muir@mnd.rs> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-12-17 22:06:31 -07:00
ReferencesCount = 8
internal/lsp: permit renaming symbols declared in other packages Also, add a test for it. Fixes golang/go#36743 Change-Id: I750ea36189a4ec5c9dc8553e4739b0238c2b29c8 Reviewed-on: https://go-review.googlesource.com/c/tools/+/216306 Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com>
2020-01-25 10:59:22 -07:00
RenamesCount = 23
tools/gopls: add cmd support for prepare_rename This change adds command line support for prepare_rename. Updates golang/go#32875 Change-Id: I7f155b9c8329c0faa26a320abab162730a7916ad GitHub-Last-Rev: 118e846420d7c56fbf0e4f416c8fd6a52e9638a8 GitHub-Pull-Request: golang/tools#188 Reviewed-on: https://go-review.googlesource.com/c/tools/+/207579 Run-TryBot: Rebecca Stambler <rstambler@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2020-02-03 23:09:33 -07:00
PrepareRenamesCount = 7
internal/lsp: add support for workspace symbol This change adds support for the LSP workspace/symbol. Unlike documentSymbol, the target is symbols that exist not only in a specific file, but also in the current or imported packages. It returns symbols whose name contains the query string of the request(case-insensitive), or all symbols if the query string is empty. However, the following is not implemented: - Setting of deprecated and containerName fields in SymbolInformation - Consideration of WorkspaceClientCapabilities - Progress support - CLI support Updates golang/go#33844 Change-Id: Id2a8d3c468084b9d44228cc6ed2ad37c4b52c405 Reviewed-on: https://go-review.googlesource.com/c/tools/+/213317 Reviewed-by: Rebecca Stambler <rstambler@golang.org> Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
2020-01-05 03:46:20 -07:00
SymbolsCount = 3
WorkspaceSymbolsCount = 2
internal/lsp: fix active parameter for incomplete parentheses I had originally thought I might be able to use exprAtPos for this, which is why I ended up eliminating that function when I saw it only had one use. One test also had to change in order to fit better with the spec. Specifically: "If [the active parameter is] omitted or the value lies outside the range of `signatures[activeSignature].parameters` it defaults to 0 if the active signature has parameters." Fixes golang/go#36766. Change-Id: I400d5b2db2985bfaa5efbcd91225151ca8b5f46a Reviewed-on: https://go-review.googlesource.com/c/tools/+/216309 Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com>
2020-01-25 16:22:03 -07:00
SignaturesCount = 23
internal/lsp: create links for golang/go#1234-style strings This change treats text of the format golang/go#1234 as a link to the Go issue tracker. This will improve the readability of TODOs that include a link to an issue, since it doesn't have to be an actual link. Change-Id: I27606ceb9cbb15bc6bfb1d7aa660d3f4fdd08739 Reviewed-on: https://go-review.googlesource.com/c/tools/+/212518 Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com>
2019-12-24 13:58:48 -07:00
LinksCount = 8
internal/lsp: fix find-implementation for promoted methods We weren't returning promoted methods as implementations when the promoted method was defined in a different package than the type implementing the interface. Fix by properly mapping the implementer types.Object to its containing source.Package. I generalized the implementations() result to just contain the implementer objects and their containing package. This allowed me to get rid of some result prep code in Implementation(). Fixes golang/go#35972. Change-Id: I867f2114c34e2ad39515ee3c8b6354c1cd35f7af Reviewed-on: https://go-review.googlesource.com/c/tools/+/210280 Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-12-05 22:34:17 -07:00
ImplementationsCount = 5
use a golden file for the expected test counts This makes it much easier to keep them up to date. It is also less fragile against accidental changes. Change-Id: If119f8527c0896d210650859960e77f3e0fa5a99 Reviewed-on: https://go-review.googlesource.com/c/tools/+/197505 Run-TryBot: Ian Cottrell <iancottrell@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Rebecca Stambler <rstambler@golang.org>
2019-09-26 10:54:25 -06:00
Copy Permalink