Closed zappy-shu closed 4 years ago
Merging #766 into master will decrease coverage by
0.11%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #766 +/- ##
==========================================
- Coverage 69.66% 69.55% -0.12%
==========================================
Files 64 64
Lines 3597 3567 -30
==========================================
- Hits 2506 2481 -25
+ Misses 760 758 -2
+ Partials 331 328 -3
Impacted Files | Coverage Δ | |
---|---|---|
internal/commands/image/rm.go | 58.62% <0%> (-3.88%) |
:arrow_down: |
types/parameters/parameters.go | 92.06% <0%> (-1.59%) |
:arrow_down: |
internal/store/bundle.go | 76.41% <0%> (-0.17%) |
:arrow_down: |
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 9a49da4...fd7984e. Read the comment docs.
I think we can close it as the bump as been done in https://github.com/docker/app/pull/748
- What I did
Updates vendor of cnab-to-oci to v0.3.0-beta2:
- How to verify it
Attempt to pull an app that does not exist. E.g.
docker app pull carolinebriaud/vapp
The error message should now be simplified to:
carolinebriaud/vapp: docker.io/carolinebriaud/vapp:latest: not found
- Description for the changelog Simplify error message returned when pulling CNAB bundle that is not found
- A picture of a cute animal (not mandatory)