Update CMakeLists.txt to use C++ standard 20 for unit_tests

Open-Agriculture / AgIsoStack-plus-plus

AgIsoStack++ is the completely free open-source C++ ISOBUS library for everyone

https://agisostack.com/

MIT License

170 stars 40 forks source link

Update CMakeLists.txt to use C++ standard 20 for unit_tests #358

Closed franz-ms-muc closed 7 months ago

franz-ms-muc commented 7 months ago

Update the CXX_STANDARD property from 14 to 20 in the unit_tests target of CMakeLists.txt.
fix https://github.com/Open-Agriculture/AgIsoStack-plus-plus/issues/357

GwnDaan commented 7 months ago

Hi, thanks for taking your time for this contribution! Unfortunately as stated in our CONTRIBUTING.md guideline;

Only features present in C++14 can be used to help provide the maximum compatibility to all "modern" compilers

by enforcing C++14 we can support a wider range of compilers (even the older, not updated to the latest standard ones). Hence updating the CXX_STANDARD to anything above 14 will violate that guideline.

Let's look for another way to solve #357