Skip to content

Fix Broken Links in Package file - #81700

Open
shail-mehta wants to merge 2 commits into
trunkfrom
fix-broken-link-in-doc
Open

Fix Broken Links in Package file#81700
shail-mehta wants to merge 2 commits into
trunkfrom
fix-broken-link-in-doc

Conversation

@shail-mehta

Copy link
Copy Markdown
Member

What?

Fixes 3 broken (404) links found on the Block Editor Handbook — Packages reference page

  • packages README
  • @wordpress/build README
  • package guidelines

Why?

While browsing the Packages reference documentation, I found that the above 3 links returned a 404 error.

How?

This PR corrects all 3 links so they point to valid, existing pages.

Testing Instructions

  • Open docs/reference-guides/packages.md file
  • See all 3 Links

Use of AI Tools

None

@shail-mehta shail-mehta self-assigned this Aug 16, 2026
@shail-mehta shail-mehta added the [Type] Developer Documentation Documentation for developers label Aug 16, 2026
@shail-mehta
shail-mehta marked this pull request as ready for review August 16, 2026 04:41
@shail-mehta
shail-mehta requested a balanced review from Copilot August 16, 2026 04:41
@github-actions

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: shail-mehta <shailu25@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 packages reference guide to point readers to newer documentation locations instead of repository README files.

Changes:

  • Replaces the Gutenberg packages README link with the contributor guide on managing packages.
  • Replaces the @wordpress/build README link with the WordPress Developer Resources page.
  • Adjusts the “package guidelines” link target within the document.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/reference-guides/packages.md Outdated
@shail-mehta
shail-mehta requested a review from t-hamano August 16, 2026 04:50
@github-actions

Copy link
Copy Markdown

Flaky tests detected in 41702c6.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/31927565140
📝 Reported tests:

adds and activates a new tab when pressing Enter at the end of a tab label in /test/e2e/specs/editor/blocks/tabs.spec.js, passed after 2 failed attempts.
Error: expect(locator).toHaveCount(expected) failed

Locator:  locator('[name="editor-canvas"]').contentFrame().getByRole('tab')
Expected: 3
Received: 2
Timeout:  5000ms

Call log:
  - Expect "toHaveCount" with timeout 5000ms
  - waiting for locator('[name="editor-canvas"]').contentFrame().getByRole('tab')
    14 × locator resolved to 2 elements
       - unexpected value "2"

    at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/editor/blocks/tabs.spec.js:146:25
Error: expect(locator).toHaveCount(expected) failed

Locator:  locator('[name="editor-canvas"]').contentFrame().getByRole('tab')
Expected: 3
Received: 2
Timeout:  5000ms

Call log:
  - Expect "toHaveCount" with timeout 5000ms
  - waiting for locator('[name="editor-canvas"]').contentFrame().getByRole('tab')
    14 × locator resolved to 2 elements
       - unexpected value "2"

    at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/editor/blocks/tabs.spec.js:146:25

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

Labels

[Type] Developer Documentation Documentation for developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants