Skip to content

docs: fix links#6394

Merged
mcollina merged 8 commits intofastify:mainfrom
Shriti507:links-fixing
Nov 23, 2025
Merged

docs: fix links#6394
mcollina merged 8 commits intofastify:mainfrom
Shriti507:links-fixing

Conversation

@Shriti507
Copy link
Contributor

I ran linkinator locally to check for the documentation errors and detected 9 broken links. I have resolved 8 of them and identified 1 false positive.

  1. I have removed Dead Projects (Ecosystem.md) The following repositories return a 404 Not Found. Since source code is no longer available, I removed them :
    fastify-normalize-request-reply
    fastify-custom-session
    fastify-git-server

  2. I have updated the links.
    Effect Schema: The standalone Effect-TS/schema repo is gone (404). The project has moved to a monorepo structure. I updated the link to the main 'https://github.com/Effect-TS/effect' repository

Vercel Template: The link in Serverless.md was outdated. I updated it to the current active template 'https://vercel.com/templates/other/fastify-on-vercel'

Unleash: Replaced the gh.getunleash.io redirect with the unleash GitHub repository URL ('https://github.com/Unleash/unleash').

  1. I have fixed the syntax and path
    Client Abort Guide: Wrapped http://localhost:3000/ in code backticks so the link checker does not attempt to ping a non-running local server.

Decorators.md : Fix the path of the Typescript Documentation

The link to https://www.bugsnag.com/ (in Ecosystem.md) fails with status 0 (Connection Reset) during automated checks . I verified manually that the link is valid and working so I left it as is.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Nov 21, 2025
@Shriti507 Shriti507 mentioned this pull request Nov 21, 2025
@jean-michelet jean-michelet changed the title Links fixing docs: fix links Nov 22, 2025
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina merged commit 012c249 into fastify:main Nov 23, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants