Skip to content

Add missing header#4763

Merged
nlohmann merged 3 commits intodevelopfrom
issue4761-missing-header
Apr 27, 2025
Merged

Add missing header#4763
nlohmann merged 3 commits intodevelopfrom
issue4761-missing-header

Conversation

@nlohmann
Copy link
Owner

@nlohmann nlohmann commented Apr 27, 2025

#4631 forgot to include <string_view>. This PR adds this include.

Closes #4761

Furthermore, fixes two warnings in the tests that made the latest GCC build fail.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann added this to the Release 3.12.1 milestone Apr 27, 2025
@github-actions github-actions bot added the S label Apr 27, 2025
@coveralls
Copy link

coveralls commented Apr 27, 2025

Coverage Status

coverage: 99.188%. remained the same
when pulling 03e869f on issue4761-missing-header
into 51a77f1 on develop.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@github-actions github-actions bot added M tests and removed S labels Apr 27, 2025
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann merged commit 6b91993 into develop Apr 27, 2025
139 of 140 checks passed
@nlohmann nlohmann deleted the issue4761-missing-header branch April 27, 2025 16:18
nlohmann added a commit that referenced this pull request Apr 27, 2025
* 🐛 add missing header

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

* 🚨 fix warning

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

* 🚨 fix warning

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

---------

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
nlohmann added a commit that referenced this pull request Apr 28, 2025
* ✅ add test for C++20 modules

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

* 🚨 fix warning

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

* Add missing header (#4763)

* 🐛 add missing header

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

* 🚨 fix warning

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

* 🚨 fix warning

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

---------

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

---------

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
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.

2 participants