cwltool icon indicating copy to clipboard operation
cwltool copied to clipboard

initial schema for a loop construct extension

Open mr-c opened this issue 3 years ago • 2 comments

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

mr-c avatar Mar 12 '22 13:03 mr-c

Codecov Report

Merging #1641 (fcdb94c) into main (0f4459c) will increase coverage by 0.17%. The diff coverage is 91.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.

codecov[bot] avatar Mar 12 '22 13:03 codecov[bot]

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

cwl-bot avatar Jun 18 '22 12:06 cwl-bot

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

lgtm-com[bot] avatar Sep 29 '22 19:09 lgtm-com[bot]

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

lgtm-com[bot] avatar Sep 29 '22 19:09 lgtm-com[bot]

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

lgtm-com[bot] avatar Sep 29 '22 20:09 lgtm-com[bot]

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

lgtm-com[bot] avatar Sep 29 '22 22:09 lgtm-com[bot]

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

lgtm-com[bot] avatar Sep 29 '22 22:09 lgtm-com[bot]

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

lgtm-com[bot] avatar Sep 30 '22 08:09 lgtm-com[bot]

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

lgtm-com[bot] avatar Oct 06 '22 17:10 lgtm-com[bot]

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

lgtm-com[bot] avatar Oct 06 '22 23:10 lgtm-com[bot]

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

lgtm-com[bot] avatar Oct 07 '22 17:10 lgtm-com[bot]

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

lgtm-com[bot] avatar Oct 08 '22 09:10 lgtm-com[bot]

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

lgtm-com[bot] avatar Oct 08 '22 10:10 lgtm-com[bot]

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

lgtm-com[bot] avatar Oct 08 '22 15:10 lgtm-com[bot]