Skip to content
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

WIP: Add bazel info starlark-environments-proto #24817

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

hauserx
Copy link
Contributor

@hauserx hauserx commented Jan 2, 2025

Added bazel info starlark-environments-proto command that can be used by LSPs to obtain list of starlark symbols per starlark environment (BUILD file, .bzl file, MODULE file, etc) from running instance of bazel.

This PR needs a lot of polishing, but decided to push it early to collect comments from interested parties on related issues - CC-ing @cameron-martin, @withered-magic

Comparison between data exported by this command and data created through bazel build ...:gen_api_proto: gist

Need decision

Dump of noticed issues that need to be fixed as well:

  • Some common rule attributes are missing documentation, documentation seems to be in HTML pages for those:
    PredefinedAttributes
  • There are two forms of links, see e.g. select:
    • <a href=\"${link common-definitions#configurable-attributes}\">
    • <a href=\"../builtins/Label.html\">
  • Missing attributes for:
    • cc_proto_library, java_binary
  • Special cases:
    • experimental_java_library_export_do_not_use
  • What to do with AutoloadSymbols which does not seem to be well supported? Like android_library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant