Skip to content

Conversation

xzyfer
Copy link
Collaborator

@xzyfer xzyfer commented Jul 22, 2025

This would be a major version bump since fastify v5 has dropped support for Node < 20 forcing this packing to raise it's minimum supported version to Node 20.

Things to note:

  • fastify v5 is using the built-in NoInfer help so Typescript needed to be updated to >= 5.4
  • rimraf v5 is written in Typescript
  • fixed the type issues instead of ignoring them
  • inlined the tsconfig because the @oly_op/tsconfig package isn't property exporting the tsconfig.json
  • updated the circleci config because it was out of date and broken

Closes #297
Closes #299
Fixes: #296

Copy link

codesandbox-ci bot commented Jul 22, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@xzyfer xzyfer changed the title Update to support fastify v5 (#300) Update to support fastify v5 Jul 22, 2025
@xzyfer xzyfer mentioned this pull request Jul 22, 2025
* Update to fastify v5

* Minimize diff

* Widen fastify support to ^5.3.0

* Fastify v5 requires Typescript >= 5.4

* Update circle matrix to reflect new support window

* Replace Node 16 and 18 references with Node 20

* Use tsx to fix postbuild script

* Try to update circleci

* Fix circleci and prettier

* Fix tsconfig resolution error and incompatible @apollo/server dependencies

* Fix prettier

* Remove @oly_op/tsconfig
@xzyfer xzyfer force-pushed the fastify-5-develop branch from 5c30509 to 79a4098 Compare August 7, 2025 04:47
@xzyfer xzyfer closed this Aug 7, 2025
@xzyfer xzyfer reopened this Aug 7, 2025
@xzyfer xzyfer force-pushed the fastify-5-develop branch from 986ad5f to 79a4098 Compare August 8, 2025 07:19
@xzyfer xzyfer merged commit fbf7bbe into main Aug 8, 2025
7 checks passed
@xzyfer xzyfer deleted the fastify-5-develop branch August 8, 2025 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for fastify 5
1 participant