Conversation
Workaround bazelbuild/bazel-skylib#568 Allows us to guarantee stardoc can run across the graph. Also: publish a docs.tar.gz file on releases, so BCR UI will render the docs.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
| load("@aspect_bazel_lib//lib:utils.bzl", bazel_lib_utils = "utils") | ||
| load("@bazel_skylib//:bzl_library.bzl", "bzl_library") | ||
| load("@bazel_lib//:bzl_library.bzl", "bzl_library") | ||
|
|
There was a problem hiding this comment.
[P1] Add bazel_lib to workspace dependencies
The change replaces all bzl_library loads with @bazel_lib//:bzl_library.bzl (see this file), yet rules_oci_dependencies()—which non‑bzlmod users are instructed to call from WORKSPACE—still only declares bazel_skylib and never creates a bazel_lib repository. Any workspace integration will now fail with no such package '@bazel_lib//...'. An http_archive for bazel_lib should be added alongside the existing dependencies to keep legacy setups working.
Useful? React with 👍 / 👎.
Needed for query --output_file
1 participant
Workaround bazelbuild/bazel-skylib#568
Allows us to guarantee stardoc can run across the graph.
Also:
publish a docs.tar.gz file on releases, so BCR UI will render the docs.