Conversation
|
doctest causes an error under clang‑22 (doctest/doctest#1042). |
|
Thanks a lot for reporting in the Since we have no urgency here, let's see when they will fix it, so we don't need a workaround like this in our 🙈 |
|
This is the only warning, so users can disable the warnings on their side without any other problems I guess. |
|
Users of FunctionalPlus probably won't even need to disable the warning, because doctest is only imported in the test files, not in the actual library headers. |
Ah yes, that makes sense. So even less urgent. |
|
But I guess we need to disable it here to make the CI of this PR pass. Or what approach would you suggest? |
|
I’m not in a hurry with the PR, and it will still take some time before Clang 22 is widely adopted. |
|
Ah, right. Yeah, then let's just wait. |
|
Quick update: Doctest fixed the problem and we now just have to wait for a new release. |
|
New release is out – with Clang 22 support 🎉. Conan package of doctest 2.5.0 is not yet available, but ticket open: conan-io/conan-center-index#29875 However, I still need to fix a macOS issue to get back to a green build: Mac OS picks up a |
|
Final review and we are good to go I hope :-) |
|
Perfect, thanks a lot! ❤️ |
No description provided.