Skip to content

update for Fastify v5 #505

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

Merged
merged 10 commits into from
Jan 28, 2024
Merged

update for Fastify v5 #505

merged 10 commits into from
Jan 28, 2024

Conversation

gurgunday
Copy link
Member

@gurgunday gurgunday changed the base branch from master to next January 24, 2024 08:16
@gurgunday
Copy link
Member Author

Looks like coverage

@gurgunday gurgunday changed the title update for v5 update for Fastify v5 Jan 24, 2024
@gurgunday gurgunday added the semver-major Issue or PR that should land as semver major label Jan 25, 2024
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 61fcbe7 into fastify:next Jan 28, 2024
@gurgunday gurgunday deleted the nex branch January 28, 2024 10:27
@gurgunday gurgunday mentioned this pull request Mar 20, 2024
4 tasks
jsumners pushed a commit that referenced this pull request Jul 15, 2024
jsumners pushed a commit that referenced this pull request Jul 15, 2024
jsumners added a commit that referenced this pull request Jul 17, 2024
* update for Fastify v5 (#505)

* fix: make sure the handler resolves in all cases (#507)

* fix error in processing file

* fix error in processing file

* add test

* fix close

* fix test

* lint

* revert change

* Revert "revert change"

This reverts commit 17ec56c.

* node 14 :(

* node 14 :(

* add file

* use filePath

* delete foo

* revert change

* remove console.log

* add test number

* increase timeout

* try fixing the error

* try fixing the error

* try reverting?

* Revert "try reverting?"

This reverts commit 35aca1e.

* 90

* fix error

* fix error

* let request aborted destroy the file

* simplify

* simplify

* ci

* update for Fastify v5 (#505)

* fix: make sure the handler resolves in all cases (#507)

* fix error in processing file

* fix error in processing file

* add test

* fix close

* fix test

* lint

* revert change

* Revert "revert change"

This reverts commit 17ec56c.

* node 14 :(

* node 14 :(

* add file

* use filePath

* delete foo

* revert change

* remove console.log

* add test number

* increase timeout

* try fixing the error

* try fixing the error

* try reverting?

* Revert "try reverting?"

This reverts commit 35aca1e.

* 90

* fix error

* fix error

* let request aborted destroy the file

* simplify

* simplify

* ci

* update fastify deps

* update for Fastify v5 (#505)

* fix: make sure the handler resolves in all cases (#507)

* fix error in processing file

* fix error in processing file

* add test

* fix close

* fix test

* lint

* revert change

* Revert "revert change"

This reverts commit 17ec56c.

* node 14 :(

* node 14 :(

* add file

* use filePath

* delete foo

* revert change

* remove console.log

* add test number

* increase timeout

* try fixing the error

* try fixing the error

* try reverting?

* Revert "try reverting?"

This reverts commit 35aca1e.

* 90

* fix error

* fix error

* let request aborted destroy the file

* simplify

* simplify

* ci

* Update index.js

Co-authored-by: Gürgün Dayıoğlu <[email protected]>
Signed-off-by: James Sumners <[email protected]>

---------

Signed-off-by: James Sumners <[email protected]>
Co-authored-by: Gürgün Dayıoğlu <[email protected]>
Co-authored-by: Gürgün Dayıoğlu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-major Issue or PR that should land as semver major
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants