BYU-Bazel / bazel-ls

A Bazel language server prototype
Apache License 2.0
10 stars 0 forks source link

Missing Syntax Format for Diagnostics or Workspace Tree on gen rules #164

Open jasonxris opened 3 years ago

jasonxris commented 3 years ago

In the bazel project there are some incorrect Diagnostics.

Travel to works fine, just the diagnostic doesn't work.


image

Here is the build file in question.

image

It seems as if you can refer to a build target by its outs field. In this case the outs has the attribute "_outs = ["builtinsbzl.zip"]" So in the given bazel build file when the API tries to find the build target _builtinsbzl.zip it can't find anything and says it is invalid, even though it is valid Bazel Syntax. The workspace tree can only find _builtins_bzlzip

So should the workspace tree parser also part "Outs" and put them in as valid build targets?

jasonxris commented 3 years ago

Same case with other out fields

image

image