cwltool icon indicating copy to clipboard operation
cwltool copied to clipboard

--leave-tmpdir includes the input staging directories

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

found while working with @RenskeW

mr-c avatar May 31 '22 10:05 mr-c

Codecov Report

Merging #1674 (909e87e) into main (741f4e5) will decrease coverage by 11.24%. The diff coverage is 0.00%.

:exclamation: Current head 909e87e differs from pull request most recent head d258967. Consider uploading reports for the commit d258967 to get more accurate results

@@             Coverage Diff             @@
##             main    #1674       +/-   ##
===========================================
- Coverage   83.70%   72.47%   -11.24%     
===========================================
  Files          44       44               
  Lines        8133     8123       -10     
  Branches     2224     2138       -86     
===========================================
- Hits         6808     5887      -921     
- Misses        848     1746      +898     
- Partials      477      490       +13     
Impacted Files Coverage Δ
cwltool/job.py 74.60% <0.00%> (-6.89%) :arrow_down:

... and 29 files with indirect coverage changes

codecov[bot] avatar May 31 '22 10:05 codecov[bot]

Sorry the delay. Tested with the example 15 from the user guide. Confirmed the directory is not deleted. Everything looking good! :+1: Thanks

image

cwltool --leave-tmpdir linkfile.cwl arguments-job.yml

kinow avatar Jun 24 '22 10:06 kinow

wanted to bump this and see if it could be merged. it really helps with debugging and checking intermediate steps in workflows

ltirrell avatar Mar 08 '23 22:03 ltirrell

sorry for another comment, but is there anything holding this back from being merged?

ltirrell avatar May 10 '23 18:05 ltirrell

sorry for another comment, but is there anything holding this back from being merged?

I think something changed in the commits after I tested it. CI builds are failing now.

kinow avatar May 10 '23 18:05 kinow

GitHub Actions and other things are broken https://www.githubstatus.com/incidents/pr3498h3qkfy

mr-c avatar May 10 '23 18:05 mr-c