Codecov Report

Merging #617 (628e689) into dev (f54a4e4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #617   +/-   ##
=======================================
  Coverage   92.75%   92.75%           
=======================================
  Files           2        2           
  Lines        2071     2071           
=======================================
  Hits         1921     1921           
  Misses        150      150           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f54a4e4...628e689. Read the comment docs.

Change looks good, I'm only wondering at the purpose of doctest::doctest although admittedly my experience with CMake is pretty limited!

It is considered a best practise to "namespace" your public facing targets. This makes them look like the the targets CMake makes available from find_package(...), like Boost::boost or Qt::Widgets.
So it became a convention a lot of projects follow. Here are some sources giving more elaborate reason:

• https://gist.github.com/mbinna/c61dbb39bca0e4fb7d1f73b0d66a4fd1#using-a-library-defined-in-the-same-cmake-tree-should-look-the-same-as-using-an-external-library
• https://cliutils.gitlab.io/modern-cmake/chapters/install.html

Thanks for your contribution!