Bump cmake version and resolve build issues

[spectre-ns]

Checklist

• [ x] The title and commit message(s) are descriptive.
• [x ] Small commits made to fix your PR have been squashed to avoid history pollution.
• [x ] Tests have been added for new features or bug fixes.
• [ x] API of new functions and classes are documented.

Description

Fix #2743
Tiny PR to silence warnings. No one should be using CMake 3.1 in this day and age

[spectre-ns]

@tdegeus we were building Windows Clang 16 in debug, and I think there is a bug (or overly pessimistic check) in the iterator checks in MSVC's STL implementation on clang in debug referenced in #2743. Normally this would be a security issue but the same code passes in MSVC on Windows presumably using the same STL, I'm inclined to think this is a library bug.

[tdegeus]

Agreed. Thanks a lot!