Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update for Fastify v5 #125

Merged
merged 2 commits into from
Mar 25, 2024
Merged

Update for Fastify v5 #125

merged 2 commits into from
Mar 25, 2024

Conversation

synapse
Copy link
Contributor

@synapse synapse commented Mar 19, 2024

Ref: fastify/fastify#5116

Checklist

CC @simoneb

@simoneb simoneb changed the base branch from master to next March 19, 2024 16:30
@simoneb
Copy link

simoneb commented Mar 19, 2024

created and rebased

Copy link
Member

@gurgunday gurgunday left a comment

Choose a reason for hiding this comment

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

disable-coverage seems to be unnecessary here as coverage is at 100%

@synapse synapse requested a review from gurgunday March 25, 2024 08:43
Copy link
Member

@gurgunday gurgunday left a comment

Choose a reason for hiding this comment

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

lgtm

@gurgunday gurgunday merged commit b7b8e64 into fastify:next Mar 25, 2024
14 checks passed
jsumners pushed a commit that referenced this pull request Jun 21, 2024
* upgrade for fastify 5

* removed disable coverage
jsumners pushed a commit that referenced this pull request Jun 21, 2024
* upgrade for fastify 5

* removed disable coverage
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants