Skip to content

Conversation

@paveltomin
Copy link
Collaborator

introduced in #3364

@paveltomin
Copy link
Collaborator Author

@CusiniM can you approve please?

@paveltomin paveltomin added ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI ci: run code coverage enables running of the code coverage CI jobs and removed flag: ready for review labels Oct 25, 2024
@codecov
Copy link

codecov bot commented Oct 25, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 56.79%. Comparing base (9ee88f7) to head (2ebe1f0).
Report is 75 commits behind head on develop.

Files with missing lines Patch % Lines
...Flow/ThermalCompositionalMultiphaseBaseKernels.hpp 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3399   +/-   ##
========================================
  Coverage    56.79%   56.79%           
========================================
  Files         1076     1076           
  Lines        95739    95739           
========================================
  Hits         54375    54375           
  Misses       41364    41364           

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

@paveltomin paveltomin added the flag: no rebaseline Does not require rebaseline label Oct 26, 2024
@paveltomin paveltomin merged commit cd8175d into develop Oct 26, 2024
24 of 26 checks passed
@paveltomin paveltomin deleted the pt/residualnorm-bugfix branch October 26, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: run code coverage enables running of the code coverage CI jobs ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI flag: no rebaseline Does not require rebaseline type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants