Skip to content

contributing containerd.io website code#630

Closed
chanezon wants to merge 4 commits intocontainerd:masterfrom
chanezon:master
Closed

contributing containerd.io website code#630
chanezon wants to merge 4 commits intocontainerd:masterfrom
chanezon:master

Conversation

@chanezon
Copy link
Copy Markdown
Contributor

It's a simple jekyll website.

@GordonTheTurtle
Copy link
Copy Markdown

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

$ git clone -b "master" [email protected]:chanezon/containerd.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842353405296
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.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 15, 2017

Codecov Report

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

@@           Coverage Diff           @@
##           master     #630   +/-   ##
=======================================
  Coverage   65.49%   65.49%           
=======================================
  Files           3        3           
  Lines         455      455           
=======================================
  Hits          298      298           
  Misses         95       95           
  Partials       62       62

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 92c7ed0...1222582. Read the comment docs.

@chanezon chanezon closed this Mar 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants