snitch-org / snitch

Lightweight C++20 testing framework.
Boost Software License 1.0
252 stars 7 forks source link

Lower required CMake version for library build #121

Closed cschreib closed 11 months ago

cschreib commented 11 months ago

This help ConanCenter, which is running 3.18.2. We actually need 3.20, but only for building the tests. For building the library, 3.14 is enough.

Also closes #120 and turns on CMake warnings as errors in CI, so we catch these in the future.

codecov[bot] commented 11 months ago

Codecov Report

Merging #121 (81324bd) into main (8b28fad) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 81324bd differs from pull request most recent head fa10789. Consider uploading reports for the commit fa10789 to get more accurate results

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/cschreib/snitch/pull/121/graphs/tree.svg?width=650&height=150&src=pr&token=X422DE81PN&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Corentin+Schreiber)](https://app.codecov.io/gh/cschreib/snitch/pull/121?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Corentin+Schreiber) ```diff @@ Coverage Diff @@ ## main #121 +/- ## ======================================= Coverage 93.67% 93.67% ======================================= Files 27 27 Lines 1597 1597 ======================================= Hits 1496 1496 Misses 101 101 ``` ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/cschreib/snitch/pull/121?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Corentin+Schreiber). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Corentin+Schreiber) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/cschreib/snitch/pull/121?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Corentin+Schreiber). Last update [8b28fad...fa10789](https://app.codecov.io/gh/cschreib/snitch/pull/121?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Corentin+Schreiber). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Corentin+Schreiber).