search

common-workflow-language / cwltool

Common Workflow Language reference implementation
https://cwltool.readthedocs.io/
Apache License 2.0
324 stars 225 forks source link

Prov data input output #1989

Open mr-c opened 3 months ago

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 59.13978% with 38 lines in your changes missing coverage. Please review.

Project coverage is 75.69%. Comparing base (ca7f768) to head (4d41ca4). Report is 1 commits behind head on main.

Files Patch % Lines
cwltool/cwlprov/ro.py 43.18% 20 Missing and 5 partials :warning:
cwltool/cwlprov/__init__.py 53.84% 6 Missing :warning:
cwltool/cwlprov/provenance_profile.py 40.00% 6 Missing :warning:
cwltool/cwlprov/writablebagfile.py 75.00% 0 Missing and 1 partial :warning:

:exclamation: There is a different number of reports uploaded between BASE (ca7f768) and HEAD (4d41ca4). Click for more details.

HEAD has 11 uploads less than BASE | Flag | BASE (ca7f768) | HEAD (4d41ca4) | |------|------|------| ||16|5|
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1989 +/- ## ========================================== - Coverage 83.83% 75.69% -8.14% ========================================== Files 46 46 Lines 8244 8295 +51 Branches 2190 2105 -85 ========================================== - Hits 6911 6279 -632 - Misses 855 1433 +578 - Partials 478 583 +105 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jjkoehorst commented 2 months ago

@mr-c I think I fixed the code make cleanup could you have a look at the other tests? Not sure if they fail due to this implementation.