search

lefticus / cpp_weekly_game_project

The Unlicense
107 stars 22 forks source link

Ci fix #17

Closed ddalcino closed 4 years ago

ddalcino commented 4 years ago

I think this fixes the CI build so far. It does a lot though, so it may need to be split up:

  1. Bump GCC to 10
  2. Bump Clang to 10
  3. Run Conan using Python 3
  4. Disable unknown-attribute warnings for Clang
  5. Include <optional> header in Input.hpp
codecov-commenter commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e80871c). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #17   +/-   ##
=========================================
  Coverage          ?   10.68%           
=========================================
  Files             ?        5           
  Lines             ?      131           
  Branches          ?        0           
=========================================
  Hits              ?       14           
  Misses            ?      117           
  Partials          ?        0

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 e80871c...d2b2411. Read the comment docs.

ddalcino commented 4 years ago

I think that this project has changed sufficiently that these changes will no longer fix the CI build. While some of this PR may be useful for getting a working build, it is insufficient, and much of it is unnecessary.

Closed.