Skip to content

Conversation

@duvvuvenkataramana
Copy link

[Describe your pull request here. Please read the text below the line and make sure you follow the checklist.]

  • The changes are described in detail, both the what and why.
  • If applicable, an existing issue is referenced.
  • The Code coverage remained at 100%. A test case for every new line of code.
  • If applicable, the documentation is updated.
  • The source code is amalgamated by running make amalgamate.

Read the Contribution Guidelines for detailed information.

Signed-off-by: duvvuvenkataramana <venkataramana2048@gmail.com>
@duvvuvenkataramana
Copy link
Author

@nlohmann Review and merge

@coveralls
Copy link

coveralls commented Nov 24, 2025

Coverage Status

coverage: 99.191%. remained the same
when pulling 7e8e21b on duvvuvenkataramana:patch-2
into a0e9fb1 on nlohmann:develop.

Copy link
Owner

@nlohmann nlohmann left a comment

Choose a reason for hiding this comment

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

Just a formatting issue. Otherwise looks good.

Signed-off-by: Niels Lohmann <niels.lohmann@gmail.com>
Copy link
Owner

@nlohmann nlohmann left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@nlohmann nlohmann added this to the Release 3.12.1 milestone Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants