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

Fix examples #746

Merged
merged 1 commit into from Nov 18, 2019
Merged

Fix examples #746

merged 1 commit into from Nov 18, 2019

Conversation

carolinebriaud
Copy link
Contributor

- What I did
Update README for docker/app/examples.
Fix hello-world example.
Remove cnab-helm example.

- How to verify it
Read the files

- A picture of a cute animal (not mandatory)
download

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "fix-examples" git@github.com:carolinebriaud/app.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354324912
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Copy link
Member

@chris-crone chris-crone 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 15, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #746   +/-   ##
=======================================
  Coverage   68.98%   68.98%           
=======================================
  Files          63       63           
  Lines        3440     3440           
=======================================
  Hits         2373     2373           
  Misses        749      749           
  Partials      318      318

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 d7d8312...24944b1. Read the comment docs.

Fix hello-world example.
Remove cnab-helm 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

@ndeloof ndeloof merged commit a57a93e into docker:master Nov 18, 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.

None yet

5 participants