codecov-commenter
commented
1 year ago Codecov Report
Merging #729 (0c43a65) into develop (f58dc20) will decrease coverage by
0.17%. The diff coverage isn/a.:exclamation: Current head 0c43a65 differs from pull request most recent head 6a32137. Consider uploading reports for the commit 6a32137 to get more accurate results
:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@ Coverage Diff @@
## develop #729 +/- ##
===========================================
- Coverage 87.00% 86.84% -0.17%
===========================================
Files 37 37
Lines 1655 1649 -6
Branches 126 0 -126
===========================================
- Hits 1440 1432 -8
- Misses 215 217 +2 | Impacted Files | Coverage Δ | |
|---|---|---|
| src/game.cpp | 95.55% <ø> (-1.25%) |
:arrow_down: |
| src/coordinate.h | 100.00% <0.00%> (ø) |
|
| src/read_only.h | 66.66% <0.00%> (+33.33%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
The test for #478 was only partially implemented, so I resurrected it. Although save-states may not be the most important now, I am happy they are part of a test.
The test for #722 was outdated, as a return type has changed from int to string, and the final test had a missing
().