Skip to content

Conversation

@mr-c
Copy link
Member

@mr-c mr-c commented Sep 4, 2021

WARNING  salad:load_tool.py:381 URI prefix '${ return ["http' of '${ return ["http://edamontology.org/format_2330", 42];}
' not recognized, are you missing a $namespaces section?
WARNING  salad:load_tool.py:381 URI prefix '${ return ["http' of '${ return ["http://edamontology.org/format_2330", 42];}
' not recognized, are you missing a $namespaces section?
WARNING  salad:load_tool.py:381 URI prefix '${return "http' of '${return "http://edamontology.org/format_2330";}
' not recognized, are you missing a $namespaces section?
WARNING  salad:load_tool.py:381 URI prefix '${return "http' of '${return "http://edamontology.org/format_2330";}
' not recognized, are you missing a $namespaces section?

@mr-c mr-c changed the title wip more expr parsing tests Sep 4, 2021
@mr-c mr-c marked this pull request as draft September 4, 2021 15:21
@codecov
Copy link

codecov bot commented Sep 4, 2021

Codecov Report

Merging #1507 (982586c) into main (6af9932) will decrease coverage by 10.99%.
The diff coverage is n/a.

❗ Current head 982586c differs from pull request most recent head 6b0412f. Consider uploading reports for the commit 6b0412f to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##             main    #1507       +/-   ##
===========================================
- Coverage   66.45%   55.46%   -11.00%     
===========================================
  Files          93       47       -46     
  Lines       16541     8294     -8247     
  Branches     4390     2160     -2230     
===========================================
- Hits        10992     4600     -6392     
+ Misses       4401     3147     -1254     
+ Partials     1148      547      -601     
Impacted Files Coverage Δ
cwltool/utils.py
cwltool/cuda.py
cwltool/builder.py
cwltool/pathmapper.py
cwltool/checker.py
cwltool/job.py
cwltool/cwlviewer.py
cwltool/context.py
cwltool/sandboxjs.py
cwltool/cwlrdf.py
... and 36 more

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 6af9932...6b0412f. Read the comment docs.

@mr-c mr-c marked this pull request as ready for review February 2, 2022 09:51
@mr-c mr-c merged commit 3c3a1e5 into main Feb 2, 2022
@mr-c mr-c deleted the more_expr_tests branch February 2, 2022 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants