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.

Sign up for GitHub

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

Jump to bottom

WIP: Replace builtin.pbtxt with one from bazel info starlark-environments-proto #102

Draft

hauserx wants to merge 2 commits into cameron-martin:master from hauserx:use_stralark_env_command

+8,310 −7,619

Contributor

hauserx commented Jan 15, 2025

Follows up WIP change in bazel that adds new bazel info command: bazelbuild/bazel#24817

Some advantages:

Global symbols are file type specific, suggesting symbols and lint are more accurate.
- Supporting .bzl, MODULE.bazel, WORKSPACE, REPO.bazel, VENDOR.bazel
Multiple symbols have additional documentation now, including multiple rules as well as some module-like symbols, like attr
No special handing for missing globals, those symbols are properly reported by bazel info command

hauserx added 2 commits

January 15, 2025 16:44


          Replace builtin.pbtxt with one from bazel info starlark-environments-…

431be44

…proto


          Use symbols relevant to file type

9ba9fd0

hauserx force-pushed the use_stralark_env_command branch from 4c0ac86 to 9ba9fd0 Compare

January 15, 2025 15:45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet