-
Notifications
You must be signed in to change notification settings - Fork 177
Conversation
Codecov Report
@@ Coverage Diff @@
## master #702 +/- ##
=======================================
Coverage 72.71% 72.71%
=======================================
Files 59 59
Lines 3042 3042
=======================================
Hits 2212 2212
Misses 556 556
Partials 274 274 Continue to review full report at Codecov.
|
243a28e
to
06c8b36
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The image argument and cnab-bundle-json
flag are mutually exclusive.
06c8b36
to
5746a0b
Compare
5746a0b
to
c1aef50
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
A new flag `--cnab-bundle-json` is added for running a CNAB bundle directly. Otherwise, a built app is ran. Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
c1aef50
to
4158c4b
Compare
- What I did
A new flag
--cnab-bundle-json
is added for running a CNAB bundle directly. Otherwise, a built app is ran.- How to verify it
Run
docker app run --cnab-bundle-json e2e/testdata/cnab-without-status/bundle.json
.- Description for the changelog
You can run a CNAB bundle with
docker app run --cnab-bundle-json BUNDLE
- A picture of a cute animal (not mandatory but encouraged)