Jump through aliases with go to definition
It seems that by default the definition LSP call will (reasonably) jump to the common pattern where folks re-export symbols. I think in the common case I would prefer it jump through that alias straight to the underlying definition.
For example on this rule:
shared_library( < - cursor somewhere on this line
name = "foo",
srcs = glob(["*.cpp"]),
...
)
And you jump to the definition, you can end up in a file like this:
load("//bazel/internal:shared_library.bzl", _shared_library = "shared_library")
shared_library = _shared_library
Then you need to do another jump to definition on the right hand side of the assignment to get through to the underlying definition.
maybe this can be what go to declaration does?
"Go to Declaration" now does this by default! You can also enable the --experimental_goto_definition_skip_re_exports flag to enable this for "Go to Definition" as well if you so like