Skip to content

Conversation

@Fdawgs
Copy link
Contributor

@Fdawgs Fdawgs commented Jun 19, 2025

Fastify moved from fastify.io to fastify.dev at the end of 2023.

fastify.io currently redirects to fastify.dev, but this may change in the future. :)

Copilot AI review requested due to automatic review settings June 19, 2025 15:27
@Fdawgs Fdawgs requested a review from a team as a code owner June 19, 2025 15:27
@github-actions github-actions bot added the JS label Jun 19, 2025
Copy link
Contributor

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

This PR updates documentation references for Fastify to reflect the new tld by replacing "fastify.io" with "fastify.dev".

  • Updated URL in Fastify documentation comments
  • Ensured consistency in referencing Fastify endpoints
Comments suppressed due to low confidence (2)

javascript/ql/lib/semmle/javascript/frameworks/Fastify.qll:2

  • The documentation URL has been updated to fastify.dev. Verify that all related documentation references are correctly updated for consistency.
 * Provides classes for working with [Fastify](https://www.fastify.dev/) applications.

javascript/ql/lib/semmle/javascript/frameworks/Fastify.qll:9

  • Ensure that this updated URL matches the intended new reference for Fastify throughout the module.
 * Provides classes for working with [Fastify](https://www.fastify.dev/) applications.

@Napalys Napalys added the no-change-note-required This PR does not need a change note label Jun 23, 2025
Copy link
Contributor

@Napalys Napalys left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution. 👍

@asgerf asgerf merged commit 93c891a into github:main Jun 23, 2025
15 of 16 checks passed
@Fdawgs Fdawgs deleted the patch-1 branch June 23, 2025 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

JS no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants