Skip to content

docs: add warning about empty string coercion with nullable types#6452

Merged
Eomm merged 10 commits into
fastify:mainfrom
ritambh888:main
Jun 28, 2026
Merged

docs: add warning about empty string coercion with nullable types#6452
Eomm merged 10 commits into
fastify:mainfrom
ritambh888:main

Conversation

@ritambh888

@ritambh888 ritambh888 commented Jan 17, 2026

Copy link
Copy Markdown
Contributor

Checklist

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jan 17, 2026
Comment thread docs/Reference/Validation-and-Serialization.md Outdated
Comment thread docs/Reference/Validation-and-Serialization.md Outdated
Comment thread docs/Reference/Validation-and-Serialization.md Outdated
@ritambh888

Copy link
Copy Markdown
Contributor Author

Hi @Eomm

I've added the regression test for the empty string coercion as discussed (commit ee83489).

It looks like the GitHub Workflows are waiting for approval to run. Could you please approve the CI run so we can verify the tests pass? I've also updated the branch to keep it current.

Thanks!

@ritambh888 ritambh888 requested a review from Eomm January 27, 2026 14:18

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ritambh888

Copy link
Copy Markdown
Contributor Author

@Eomm @mcollina I have updated the test to use the native node:assert as requested, to be compatible with the new test runner. The local tests differ slightly due to the runner change, but the logic covers the regression. Ready for re-review.

@gurgunday

Copy link
Copy Markdown
Member

@ritambh888 lint is failing

@ritambh888

ritambh888 commented Jan 30, 2026

Copy link
Copy Markdown
Contributor Author

@ritambh888 lint is failing

@gurgunday Yes, I fixed it now. Please re-run the tests.

mcollina
mcollina previously approved these changes Jan 31, 2026

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mcollina

Copy link
Copy Markdown
Member

@Eomm ptal

Comment thread docs/Reference/Validation-and-Serialization.md Outdated
Comment thread docs/Reference/Validation-and-Serialization.md Outdated
Comment thread docs/Reference/Validation-and-Serialization.md Outdated
@Fdawgs Fdawgs requested a review from Eomm March 7, 2026 09:45
ritambh888 and others added 9 commits March 13, 2026 00:44
Co-authored-by: Manuel Spigolon <behemoth89@gmail.com>
Signed-off-by: Ritam Bhattacharjee <151223264+ritambh888@users.noreply.github.com>
Co-authored-by: Manuel Spigolon <behemoth89@gmail.com>
Signed-off-by: Ritam Bhattacharjee <151223264+ritambh888@users.noreply.github.com>
Co-authored-by: Manuel Spigolon <behemoth89@gmail.com>
Signed-off-by: Frazer Smith <frazer.dev@icloud.com>
@Eomm Eomm merged commit ea454d0 into fastify:main Jun 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants