Skip to content

Update for Fastify v5#238

Merged
gurgunday merged 2 commits intofastify:nextfrom
synapse:next
Mar 22, 2024
Merged

Update for Fastify v5#238
gurgunday merged 2 commits intofastify:nextfrom
synapse:next

Conversation

@synapse
Copy link
Contributor

@synapse synapse commented Mar 21, 2024

Ref: fastify/fastify#5116

Checklist

CC @simoneb

@synapse synapse changed the base branch from next to master March 21, 2024 10:35
@synapse
Copy link
Contributor Author

synapse commented Mar 21, 2024

⚠️ there's another next branch here, can I use that one?

@synapse synapse changed the base branch from master to next March 21, 2024 10:38
Signed-off-by: Cristian Barlutiu <cristian.barlutiu@gmail.com>
@simoneb
Copy link
Contributor

simoneb commented Mar 21, 2024

not sure what's in that branch, can it be it was alredy dealt with?

@synapse
Copy link
Contributor Author

synapse commented Mar 21, 2024

Yup, saw it afterwards, I've bumped a couple more deps and changed the CI version

@simoneb
Copy link
Contributor

simoneb commented Mar 21, 2024

ok sounds good 👌

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 e15b667 into fastify:next Mar 22, 2024
jsumners pushed a commit that referenced this pull request Jun 21, 2024
Signed-off-by: Cristian Barlutiu <cristian.barlutiu@gmail.com>
jsumners added a commit that referenced this pull request Jun 23, 2024
* update for Fastify v5 (#236)

* update for v5

* add pre commit

* upgrade for fastify 5 (#238)

Signed-off-by: Cristian Barlutiu <cristian.barlutiu@gmail.com>

* update for Fastify v5 (#236)

* update for v5

* add pre commit

* upgrade for fastify 5 (#238)

Signed-off-by: Cristian Barlutiu <cristian.barlutiu@gmail.com>

* update deps and tests

---------

Signed-off-by: Cristian Barlutiu <cristian.barlutiu@gmail.com>
Co-authored-by: Gürgün Dayıoğlu <hey@gurgun.day>
Co-authored-by: Cristian Barlutiu <cristian.barlutiu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants