Skip to content
This repository was archived by the owner on Nov 26, 2024. It is now read-only.

The game use SFML minimally#71

Open
richelbilderbeek wants to merge 8 commits into
developfrom
issue_70
Open

The game use SFML minimally#71
richelbilderbeek wants to merge 8 commits into
developfrom
issue_70

Conversation

@richelbilderbeek

@richelbilderbeek richelbilderbeek commented Feb 21, 2024

Copy link
Copy Markdown
Member

Fixes #70

About the gcov things: game.pro is modified to allow (1) all team members to work, and (2) allow GitHub Action to actually measure the code coverage.

About LOGIC_ONLY: this define LOGIC_ONLY determines if the game is run visually. On GitHub Action, showing a window will crash the program (as there is no screen to display it on).

We did not fix the lint :-)

If anything is unclear, or you are just wondering, hit me and Theo with your questions and feedback :-)

@codecov

codecov Bot commented Feb 21, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (f9606e3) to head (13f18e7).
Report is 43 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff            @@
##           develop       #71   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          104       102    -2     
  Branches         1         1           
=========================================
- Hits           104       102    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants