docker / app

Make your Docker Compose applications reusable, and share them on Docker Hub
Apache License 2.0
1.57k stars 176 forks source link

Fix handling short id in docker app commands #717

Closed jcsirot closed 4 years ago

jcsirot commented 4 years ago

- What I did This PR add the possibility to use short id in docker app commands (app image tag , app image remove, app run...)

- How I did it The bundle store interface now exposes a LookUp function which convert a string (an id, short id, tagged or digested reference) into a reference.Reference object if it exists in the store. A map of ID to []reference.Reference is also maintained by the bundle store implementation.

- How to verify it Some unit tests and e2e tests have been added

- A picture of a cute animal (not mandatory but encouraged)

7702124

codecov[bot] commented 4 years ago

Codecov Report

Merging #717 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #717   +/-   ##
=======================================
  Coverage   72.45%   72.45%           
=======================================
  Files          59       59           
  Lines        3438     3438           
=======================================
  Hits         2491     2491           
  Misses        612      612           
  Partials      335      335

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 6267dc4...280f562. Read the comment docs.