Closed jcschulz closed 9 months ago
Merging #131 into master will decrease coverage by
3.12%
. The diff coverage is75.00%
.
@@ Coverage Diff @@
## master #131 +/- ##
==========================================
- Coverage 73.20% 70.07% -3.13%
==========================================
Files 133 134 +1
Lines 7863 8451 +588
==========================================
+ Hits 5756 5922 +166
- Misses 2107 2529 +422
Impacted Files | Coverage Δ | |
---|---|---|
src/general/mppequil.cpp | 27.79% <75.00%> (ø) |
|
src/transport/Transport.h | 70.00% <0.00%> (-30.00%) |
:arrow_down: |
src/thermo/Thermodynamics.h | 75.00% <0.00%> (-6.82%) |
:arrow_down: |
src/transport/CollisionDB.h | 97.61% <0.00%> (+0.05%) |
:arrow_up: |
src/thermo/Thermodynamics.cpp | 37.26% <0.00%> (+0.42%) |
:arrow_up: |
src/thermo/StateModel.h | 71.83% <0.00%> (+1.40%) |
:arrow_up: |
src/utilities/StringUtils.cpp | 78.84% <0.00%> (+5.76%) |
:arrow_up: |
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 3f9a791...b94020e. Read the comment docs.
@grgbellasvki can you have a look at what the conflicts are?
@jcschulz I rebased your branch on latest master. There were no conflicts, but you may want to double check that it still behaves as you expect.
I was not able to update this pull request so I have opened #249. When that is merged I will close this PR.
See Issue #130