Skip to content
This repository has been archived by the owner on Jun 13, 2021. It is now read-only.

Update CNAB-simple example to reflect the UX changes #754

Merged
merged 2 commits into from
Nov 20, 2019

Conversation

carolinebriaud
Copy link
Contributor

@carolinebriaud carolinebriaud commented Nov 19, 2019

Signed-off-by: Caroline Briaud caroline.briaud@docker.com

- What I did
Update CNAB-simple example to reflect the UX changes.
Add final step (remove running app) to hello-world example.

- How to verify it
Check the README file

- A picture of a cute animal (not mandatory)
34053984270_99b1cb12c2_b

Signed-off-by: Caroline Briaud <caroline.briaud@docker.com>
examples/cnab-simple/README.md Outdated Show resolved Hide resolved
examples/cnab-simple/README.md Outdated Show resolved Hide resolved
examples/cnab-simple/README.md Outdated Show resolved Hide resolved
examples/cnab-simple/README.md Show resolved Hide resolved
Add final step (remove running app) to hello-world example.

Signed-off-by: Caroline Briaud <caroline.briaud@docker.com>
Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Nov 20, 2019

Codecov Report

Merging #754 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #754     +/-   ##
=========================================
- Coverage   69.35%   69.25%   -0.1%     
=========================================
  Files          63       63             
  Lines        3462     3428     -34     
=========================================
- Hits         2401     2374     -27     
+ Misses        742      738      -4     
+ Partials      319      316      -3
Impacted Files Coverage Δ
internal/inspect/inspect.go 89.2% <0%> (-0.66%) ⬇️
internal/commands/image/inspect.go 69.56% <0%> (+2.21%) ⬆️
types/parameters/parameters.go 96.82% <0%> (+4.76%) ⬆️

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 6c2b173...ede60f1. Read the comment docs.

@rumpl rumpl merged commit 355ab4f into docker:master Nov 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants