Skip to content

Conversation

@abishekgiri
Copy link

What does this PR do?

This PR adds missing load() statements to documentation examples that reference
rules or symbols without showing how they are imported.

Several examples currently assume implicit knowledge of where rules are defined,
which can be confusing for new users.


What’s included?

  • Added the appropriate load() statements to affected examples
  • No behavioral or semantic changes to the documented examples
  • Improves copy-paste usability and clarity

Why is this useful?

Without explicit load() statements, examples may not work as written and can
lead to confusion for users learning Bazel rules and macros.

This change makes the documentation more self-contained and accurate.


Verification

  • Manually reviewed updated examples for correctness and consistency
  • Confirmed all added load() statements match the rules used in each example

Fixes #16001

@github-actions github-actions bot added team-Documentation Documentation improvements that cannot be directly linked to other team labels awaiting-review PR is awaiting review from an assigned reviewer labels Jan 26, 2026
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds missing load() statements to code examples in the documentation, which improves their accuracy and makes them easier for users to copy and use directly. The changes are correct and well-targeted. No issues were found during the review.

@iancha1992 iancha1992 added the team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website label Jan 27, 2026
@meteorcloudy
Copy link
Member

We should update those examples to use MODULE.bazel

@ahumesky

@fweikert fweikert added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally team-Documentation Documentation improvements that cannot be directly linked to other team labels team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation issue: General Rules and other pages lack "load" statements

4 participants