Skip to content

Fix conversion to std::optional#5052

Merged
nlohmann merged 3 commits intodevelopfrom
issue5046
Jan 13, 2026
Merged

Fix conversion to std::optional#5052
nlohmann merged 3 commits intodevelopfrom
issue5046

Conversation

@nlohmann
Copy link
Copy Markdown
Owner

Closes #5046

The fix proposed by @gregmarr in #5046 (comment) fixes the reported regression.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann added the review needed It would be great if someone could review the proposed changes. label Jan 12, 2026
@coveralls
Copy link
Copy Markdown

coveralls commented Jan 12, 2026

Coverage Status

coverage: 99.192%. remained the same
when pulling cd1f291 on issue5046
into 457bc28 on develop.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann removed the review needed It would be great if someone could review the proposed changes. label Jan 13, 2026
@nlohmann
Copy link
Copy Markdown
Owner Author

There is still a failing test on AppVeyor, but that should be trivial to fix.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann added this to the Release 3.12.1 milestone Jan 13, 2026
@nlohmann nlohmann merged commit e3014f1 into develop Jan 13, 2026
143 checks passed
@nlohmann nlohmann deleted the issue5046 branch January 13, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

implicit conversion of return json to std::optional no longer implicit

3 participants