Skip to content

fix: skip docs.mergify.com in broken link checker#10424

Open
jd wants to merge 1 commit intomainfrom
devs/jd/worktree-404-changelog/Ic66aa04780d9daacca73ca8c2ea9237ce52fca92
Open

fix: skip docs.mergify.com in broken link checker#10424
jd wants to merge 1 commit intomainfrom
devs/jd/worktree-404-changelog/Ic66aa04780d9daacca73ca8c2ea9237ce52fca92

Conversation

@jd
Copy link
Member

@jd jd commented Mar 5, 2026

The linkinator check fails on every new page PR because canonical/og:url
tags point to docs.mergify.com where the page doesn't exist yet. These
links are redundant since all internal pages are already validated via
the local dist/ crawl.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

The linkinator check fails on every new page PR because canonical/og:url
tags point to docs.mergify.com where the page doesn't exist yet. These
links are redundant since all internal pages are already validated via
the local dist/ crawl.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Change-Id: Ic66aa04780d9daacca73ca8c2ea9237ce52fca92
Claude-Session-Id: 1443e6a7-cbc3-4e10-8b04-a6e617b3eaa9
Copilot AI review requested due to automatic review settings March 5, 2026 21:16
@mergify mergify bot had a problem deploying to Mergify Merge Protections March 5, 2026 21:16 Failure
@mergify
Copy link
Contributor

mergify bot commented Mar 5, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🔴 👀 Review Requirements

This rule is failing.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🔴 🔎 Reviews

This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@mergify mergify bot requested a review from a team March 5, 2026 21:19
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the docs site’s CI broken-link check to avoid failing PRs where canonical/og:url point at docs.mergify.com before the page is deployed.

Changes:

  • Extend the check:links linkinator command to skip https://docs.mergify.com URLs during the dist/ crawl.

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

@mergify mergify bot requested a review from a team March 5, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants