Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update for Fastify v5 #1126

Merged
merged 1 commit into from
Mar 23, 2024
Merged

Update for Fastify v5 #1126

merged 1 commit into from
Mar 23, 2024

Conversation

synapse
Copy link
Contributor

@synapse synapse commented Mar 21, 2024

Ref: fastify/fastify#5116

⚠️ got dependency blocked at v11.8.6. Above that it breaks since they've also updated dependencies and require node v20 or above.

Checklist

CC @simoneb

@simoneb simoneb changed the base branch from main to next March 21, 2024 10:55
@simoneb
Copy link
Contributor

simoneb commented Mar 21, 2024

done

@gurgunday gurgunday merged commit 6651701 into fastify:next Mar 23, 2024
14 checks passed
mcollina added a commit that referenced this pull request Sep 11, 2024
* upgrade for fastify 5 (#1126)

* fixup

Signed-off-by: Matteo Collina <hello@matteocollina.com>

* fixup]

Signed-off-by: Matteo Collina <hello@matteocollina.com>

* fixup

Signed-off-by: Matteo Collina <hello@matteocollina.com>

* fixup

Signed-off-by: Matteo Collina <hello@matteocollina.com>

* fixup

Signed-off-by: Matteo Collina <hello@matteocollina.com>

* chore: fix type issues in Updates for Fastify v5 PR (#1198)

* chore: fix type issues

* lint

* Fix linting

I'll investigate why this rule needs to be disabled when I get a chance. 

Signed-off-by: Rob Walker <hi@robwalker.co>

* linting

---------

Signed-off-by: Rob Walker <hi@robwalker.co>

* Update src/session-managers/SecureSessionManager.ts

Co-authored-by: Dan Castillo <dan.castillo@jasper.ai>
Signed-off-by: Matteo Collina <matteo.collina@gmail.com>

---------

Signed-off-by: Matteo Collina <hello@matteocollina.com>
Signed-off-by: Rob Walker <hi@robwalker.co>
Signed-off-by: Matteo Collina <matteo.collina@gmail.com>
Co-authored-by: Cristian Barlutiu <cristian.barlutiu@gmail.com>
Co-authored-by: Gürgün Dayıoğlu <hey@gurgun.day>
Co-authored-by: Rob Walker <hi@robwalker.co>
Co-authored-by: Dan Castillo <dan.castillo@jasper.ai>
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.

3 participants