search

cpp-best-practices / gui_starter_template

A template CMake project to get you started with C++ and tooling
The Unlicense
2.5k stars 447 forks source link

ci: use setup-cpp to set up all compilers and tools for all the CI services (GitHub, Gitlab, etc.) #183

Closed aminya closed 2 years ago

aminya commented 2 years ago

https://github.com/aminya/setup-cpp

use setup-cpp to set up all compilers and tools for all the CI services (GitHub, Gitlab, etc.). This includes different compilers and platforms:

image

codecov-commenter commented 2 years ago

Codecov Report

Merging #183 (96b91c2) into main (ffb2784) will decrease coverage by 1.90%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
- Coverage   73.33%   71.42%   -1.91%     
==========================================
  Files           3        3              
  Lines          30       28       -2     
  Branches       16       14       -2     
==========================================
- Hits           22       20       -2     
- Misses          7        8       +1     
+ Partials        1        0       -1
Flag Coverage Δ
Linux 9.52% <ø> (-26.84%) :arrow_down:
Windows 100.00% <ø> (ø)
macOS ∅ <ø> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
test/tests.cpp 90.90% <0.00%> (-9.10%) :arrow_down:
test/constexpr_tests.cpp 90.90% <0.00%> (-9.10%) :arrow_down:
src/main.cpp 0.00% <0.00%> (ø)

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 ffb2784...96b91c2. Read the comment docs.

lefticus commented 2 years ago

should we drop appveyor? Does it make sense to keep maintaining it when github actions are so easy to use?

aminya commented 2 years ago

should we drop appveyor? Does it make sense to keep maintaining it when github actions are so easy to use?

I agree. I removed it. You should also disable its app from the settings, so it does not error here.