fix: update broken registry.bazel.build/docs URLs - #2999
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c2c573b35b
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
bc6fa4f to
62a69ae
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 62a69aef71
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
2fa6bcf to
87656ad
Compare
The registry.bazel.build/docs/<name> URL pattern returns 404. The new correct URL format is registry.bazel.build/modules/<name>/latest/docs. For file-level anchors, replaced with explicit paths: #npm-extensions-bzl -> /npm/extensions.bzl #lint-eslint-bzl (aspect_rules_lint) -> /lint/eslint.bzl For symbol-level anchors: #function-npm_import -> /npm/extensions.bzl/npm #module_extension-npm_import -> /npm/extensions.bzl/npm #module_extension-translate_lock -> /npm/extensions.bzl/pnpm #tag-npm-npm_translate_lock -> /npm/extensions.bzl/npm #function-jq (jq.bzl) -> /jq/jq.bzl/jq #function-patch (bazel_lib) -> /lib/repo_utils.bzl/patch #provider-directorypathinfo (bazel_lib) -> /lib/directory_path.bzl/DirectoryPathInfo
87656ad to
ad4d913
Compare
Summary
The
registry.bazel.build/docs/<name>URL pattern returns 404 and is no longer valid.The correct URL format is now
registry.bazel.build/modules/<name>/latest/docs.This was identified across multiple Bazel rule repositories — see hermeticbuild/rules_rs#235 for the original fix.
Changes
Replaced broken
/docs/<name>URLs and anchor-based fragments with explicit paths:File-level anchors:
#npm-extensions-bzl→/npm/extensions.bzl#lint-eslint-bzl(aspect_rules_lint) →/lint/eslint.bzlSymbol-level anchors:
#function-npm_import→/npm/extensions.bzl/npm#module_extension-npm_import→/npm/extensions.bzl/npm#module_extension-translate_lock→/npm/extensions.bzl/pnpm#tag-npm-npm_translate_lock→/npm/extensions.bzl/npm#function-jq(jq.bzl) →/jq/jq.bzl/jq#function-patch(bazel_lib) →/lib/repo_utils.bzl/patch#provider-directorypathinfo(bazel_lib) →/lib/directory_path.bzl/DirectoryPathInfo