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

BREAKING CHANGE: fastify v5 #261

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

ilteoood
Copy link
Contributor

@ilteoood ilteoood commented Nov 20, 2024

  • support for fastify v5
  • migration to node test runner

Due to the structure of the tests I've not been able to run them locally

Note: this contains a breaking change

Copy link

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

simoneb
simoneb previously approved these changes Nov 21, 2024
@simoneb
Copy link
Member

simoneb commented Nov 21, 2024

@ilteoood CI is failing, can you take a look?

@ilteoood
Copy link
Contributor Author

@ilteoood CI is failing, can you take a look?

Sure thing

@ilteoood ilteoood changed the title breaking: fastify v5 BREAKING CHANGE: fastify v5 Nov 21, 2024
@ilteoood ilteoood force-pushed the breakling/support-fastify-v5 branch from 682bdc5 to 1526755 Compare November 21, 2024 16:27
@ilteoood ilteoood force-pushed the breakling/support-fastify-v5 branch from 0f33c15 to cbd25be Compare November 21, 2024 16:30
@ilteoood
Copy link
Contributor Author

@ilteoood CI is failing, can you take a look?

The PR is failing because it doesn't have access to the vars to be properly executed. Instead of a fork, it should be a normal PR from this repo in order to access them.

I tried also running it locally, but the credentials on bitwarden seems to be expired. What if we merge it and see the outcome on the main branch?

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@simoneb
Copy link
Member

simoneb commented Nov 21, 2024

@ilteoood github seems to be having some issues with this repo, try maybe force pushing to trigger another sync

@ilteoood ilteoood force-pushed the breakling/support-fastify-v5 branch from 1526755 to 3cda2a7 Compare November 22, 2024 07:00
@ilteoood ilteoood requested a review from simoneb November 22, 2024 07:04
@simoneb simoneb merged commit 44183a7 into nearform:master Nov 22, 2024
1 of 4 checks passed
@github-actions github-actions bot mentioned this pull request Nov 22, 2024
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.

2 participants