Closed bgloyer closed 4 years ago
Merging #15 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #15 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 1 1
=========================================
Hits 1 1
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 ae522d3...f513d74. Read the comment docs.
@bgloyer good catch! Thank you very much for both your contribution and your appreciation!
Closes #14
Not sure if you want all of these, but I changed it to use ${CMAKE_PROJECT_NAME} for the project.
Also mentioned that SourcesAndHeaders.cmake should be changed.
Nice template.
Pull Request Readiness Checklist
See details at CONTRIBUTING.md.
[x] I agree to contribute to the project under modern-cpp-template (Unlicense) License.
[x] To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with modern-cpp-template
[x] The PR is proposed to proper branch
[x] There is reference to original bug report and related work
[x] There is accuracy test, performance test and test data in the repository, if applicable
[x] The feature is well documented and sample code can be built with the project CMake