Skip to content

Conversation

@ilteoood
Copy link
Contributor

@ilteoood ilteoood commented Nov 7, 2025

Closes #850
Closes #845
Closes #559

Checklist

@ilteoood ilteoood force-pushed the breaking/remove-dotenv branch from 02fbb30 to c12e7ee Compare November 7, 2025 16:43
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
Copy link
Member

mcollina commented Nov 7, 2025

CI is red

@ilteoood ilteoood marked this pull request as draft November 7, 2025 17:03
@ilteoood ilteoood marked this pull request as ready for review November 7, 2025 17:13
@ilteoood
Copy link
Contributor Author

ilteoood commented Nov 7, 2025

CI is red

My bad @mcollina, forgot to reset the stub

@ilteoood ilteoood requested a review from Eomm November 7, 2025 17:22
@Eomm Eomm merged commit f472d6d into fastify:main Nov 8, 2025
14 checks passed
@ilteoood ilteoood deleted the breaking/remove-dotenv branch November 8, 2025 10:23
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.

Consider removing dotenv as a dependency

3 participants