whole-tale / girder_wholetale

Girder plugin providing basic Whole Tale functionality
BSD 3-Clause "New" or "Revised" License
3 stars 5 forks source link

Include repo2docker_version on import #549

Closed craig-willis closed 2 years ago

craig-willis commented 2 years ago

Problem:

Importing a tale doesn't include the repo2docker_version used.

Approach:

Add support for imageInfo["repo2docker_version"] if present during import.

Test:

codecov[bot] commented 2 years ago

Codecov Report

Merging #549 (7f84ade) into master (04e52fc) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #549      +/-   ##
==========================================
+ Coverage   92.80%   92.81%   +0.01%     
==========================================
  Files          60       60              
  Lines        4738     4745       +7     
==========================================
+ Hits         4397     4404       +7     
  Misses        341      341              
Impacted Files Coverage Δ
server/models/tale.py 98.09% <ø> (ø)
server/lib/manifest_parser.py 87.74% <100.00%> (+0.57%) :arrow_up:

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 04e52fc...7f84ade. Read the comment docs.