TINYDOC-3514 - Narrow the tiny.cloud link-check exclusion to the documentation path v5 - #4336
Merged
Merged
Conversation
kemister85
requested review from
MitchC1999,
ShiridiGandham,
TheSpyder,
metricjs and
tiny-ben-tran
and removed request for
MitchC1999,
ShiridiGandham and
tiny-ben-tran
September 1, 2026 01:09
This was referenced Sep 1, 2026
Merged
kemister85
force-pushed
the
hotfix/5/TINYDOC-3514-narrow-exclusion
branch
from
September 1, 2026 01:10
73298bc to
e2bc621
Compare
tiny-ben-tran
approved these changes
Sep 1, 2026
ArvinJ-H
approved these changes
Sep 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ticket: TINYDOC-3514
Site: Staging (CI-only change; no page content differs)
Follow-up to #4277.
.lychee.tomlexcluded the wholetiny.clouddomain, so a pull request whose only changed link is on that domain passed having checked nothing — the job reported "0 of the PR's link(s) render as anchors" and went green. Reproduced on #4332: a link to a hard 404 under/pricing/passed before this change, and fails with it (run).The exclusion was written for documentation pages a pull request adds before they are published. That covers
tiny.cloud/docs, not the marketing and product pages, where a 404 is real link rot.Changes:
.lychee.toml: replaced'^https?://([a-z0-9-]+\.)*tiny\.cloud'with three scoped rules —/docsself-links, the TinyMCE AI API base URL (404s on its root by design), andclaims.tiny.cloud(JWT claim names, not links)..lycheeignore: addedhttps://support.tiny.cloud/(403 to automated checkers, reachable in a browser), previously covered by the domain-wide pattern.Note: each branch was built and every
tiny.cloudanchor the change newly exposes was probed — 66 on/8, 60 on/7, 48 on/6, 83 on/5, none unhealthy. Applied identically to #4333 (/latest), #4334 (v7), #4335 (v6), #4336 (v5).Pre-checks:
hotfix/5/TINYDOC-3514-narrow-exclusionReview: