cwltool
cwltool copied to clipboard
initial schema for a loop construct extension
Based upon @GlassOfWhiskey 's work in https://github.com/common-workflow-language/common-workflow-language/issues/495#issuecomment-1064343639 With comments from @tetron @mr-c
Codecov Report
Merging #1641 (fcdb94c) into main (0f4459c) will increase coverage by
0.17%. The diff coverage is91.72%.
@@ Coverage Diff @@
## main #1641 +/- ##
==========================================
+ Coverage 82.17% 82.35% +0.17%
==========================================
Files 47 47
Lines 8058 8171 +113
Branches 2191 2228 +37
==========================================
+ Hits 6622 6729 +107
- Misses 947 948 +1
- Partials 489 494 +5
| Impacted Files | Coverage Δ | |
|---|---|---|
| cwltool/workflow_job.py | 86.96% <88.76%> (+0.74%) |
:arrow_up: |
| cwltool/process.py | 92.44% <90.90%> (+0.11%) |
:arrow_up: |
| cwltool/checker.py | 96.88% <100.00%> (+0.29%) |
:arrow_up: |
| cwltool/load_tool.py | 86.66% <100.00%> (ø) |
|
| cwltool/main.py | 74.36% <100.00%> (+0.06%) |
:arrow_up: |
| cwltool/workflow.py | 91.79% <100.00%> (+0.16%) |
:arrow_up: |
| cwltool/job.py | 80.67% <0.00%> (-0.40%) |
:arrow_down: |
| cwltool/command_line_tool.py | 86.92% <0.00%> (+0.40%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
This pull request has been mentioned on Common Workflow Language Discourse. There might be relevant details there:
https://cwl.discourse.group/t/loop-requirement-implementation/611/4
This pull request introduces 2 alerts when merging 27284c62307f701ec62d5a341e6792d3dca24c33 into 260df857c8a4d3afce933eb3351dd28dd924b867 - view on LGTM.com
new alerts:
- 1 for Unused import
- 1 for `__init__` method calls overridden method
This pull request introduces 2 alerts when merging 6b414ef4646a319674badde7133778c0d0b13627 into 260df857c8a4d3afce933eb3351dd28dd924b867 - view on LGTM.com
new alerts:
- 1 for Unused import
- 1 for `__init__` method calls overridden method
This pull request introduces 2 alerts when merging ffe9345bca09b4ecd47d595dd55e20925764ca8a into 260df857c8a4d3afce933eb3351dd28dd924b867 - view on LGTM.com
new alerts:
- 1 for Unused import
- 1 for `__init__` method calls overridden method
This pull request introduces 1 alert and fixes 1 when merging 27fe2bdce22a1646d782df0d616c6ccaf7065762 into 260df857c8a4d3afce933eb3351dd28dd924b867 - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging 36aa1a55e757062ff21a3f40d01acb432fbdfad5 into 260df857c8a4d3afce933eb3351dd28dd924b867 - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging 777949759aa49310201668ec2a54756ebe96d541 into 260df857c8a4d3afce933eb3351dd28dd924b867 - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging ce8b7cae43b31b6bbc687c0d4728fae3363b9c03 into 0f4459c68d7311d3d7a6b11ecac55ba956521cbf - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging cd5d2f3463b8eb395018af97e51a2ccf851649af into 0f4459c68d7311d3d7a6b11ecac55ba956521cbf - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging 017370a23c9ac2ab596a5ca76e555e058fb714fc into 0f4459c68d7311d3d7a6b11ecac55ba956521cbf - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging c7c7ad8ec1d9c921bd9dee7a9a8480ea4e248806 into 0f4459c68d7311d3d7a6b11ecac55ba956521cbf - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging 1ae0795d9bd4960e9c04b263402adfe7841e4887 into 0f4459c68d7311d3d7a6b11ecac55ba956521cbf - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable
This pull request introduces 1 alert and fixes 1 when merging fcdb94c9ae2994720253e7f15df011025f1c3c05 into 0f4459c68d7311d3d7a6b11ecac55ba956521cbf - view on LGTM.com
new alerts:
- 1 for `__init__` method calls overridden method
fixed alerts:
- 1 for Unused local variable