Skip to content

docs: fix broken Next steps link in hooks.md#528

Closed
jimpablo wants to merge 1 commit into
MoonshotAI:mainfrom
jimpablo:docs/fix-hooks-next-steps-link
Closed

docs: fix broken Next steps link in hooks.md#528
jimpablo wants to merge 1 commit into
MoonshotAI:mainfrom
jimpablo:docs/fix-hooks-next-steps-link

Conversation

@jimpablo

@jimpablo jimpablo commented Jun 8, 2026

Copy link
Copy Markdown

Problem

hooks.md "Next steps" links to config-files.md#hooks, but config-files.md has no ## hooks section. The #hooks anchor does not exist, so the link does not scroll to any section.

What changed

Removed the broken #hooks fragment and updated the description to match the style used by other pages linking to config-files.md (e.g. data-locations.md). 2 files, 2 lines changed (EN + ZH).

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works. (N/A: docs-only fix; verified with docs build.)
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

Copilot AI review requested due to automatic review settings June 8, 2026 03:39
@changeset-bot

changeset-bot Bot commented Jun 8, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 99111f9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the “Next steps” documentation links in the Hooks customization docs to point to the general configuration file reference (instead of the #hooks anchor) and adjusts the accompanying descriptions in both English and Chinese.

Changes:

  • Updated the “Configuration files/配置文件” link target from config-files.md#hooks to config-files.md
  • Revised the link descriptions to refer to overall config.toml fields rather than specifically [[hooks]]

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
docs/zh/customization/hooks.md Updates “Next steps” configuration reference link and description (ZH)
docs/en/customization/hooks.md Updates “Next steps” configuration reference link and description (EN)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

## Next steps

- [Configuration files](../configuration/config-files.md#hooks) — Full field reference for `[[hooks]]` in `config.toml`
- [Configuration files](../configuration/config-files.md) — full reference for `config.toml` fields
## Next steps

- [Configuration files](../configuration/config-files.md#hooks) — Full field reference for `[[hooks]]` in `config.toml`
- [Configuration files](../configuration/config-files.md) — full reference for `config.toml` fields
## 下一步

- [配置文件](../configuration/config-files.md#hooks) — `[[hooks]]` 在 `config.toml` 中的完整字段声明
- [配置文件](../configuration/config-files.md) — `config.toml` 各字段的完整说明
@jimpablo

jimpablo commented Jun 9, 2026

Copy link
Copy Markdown
Author

Consolidated into #578, which collects all the broken-anchor / cross-reference link fixes into a single PR for easier review. Closing this in favor of that one.

@jimpablo jimpablo closed this Jun 9, 2026
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.

2 participants