Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Conversation

@krnowak
Copy link
Collaborator

@krnowak krnowak commented Dec 2, 2015

First commit is quite large, because of the one-sentence-per-line formatting, but it mostly is not changing any text.

The rest of the commits are fixing various mistakes.

Copy link
Member

Choose a reason for hiding this comment

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

flavorS?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Bah, right you are.

@jonboulle
Copy link
Contributor

While I appreciate the one-sentence-per-line standardisation (and I think it's the right thing for now), I would be remiss if I didn't draw attention to the fact that it might imminently change, per coreos/docs#633

@jonboulle
Copy link
Contributor

While we're here we should probably also see if @joshix has some time to review.

Copy link
Contributor

Choose a reason for hiding this comment

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

rkt runs applications packaged according to the open-source [App Container Image][aci-images] specification.

@krnowak krnowak force-pushed the krnowak/docs-fixes branch from 3d1df74 to df995e9 Compare December 4, 2015 13:32
Copy link
Contributor

Choose a reason for hiding this comment

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

Headings in this file need to be ATX: # Heading.

s/Background/## Background/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is fixed, but somehow github insists on the diff not being outdated.

Copy link
Contributor

Choose a reason for hiding this comment

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

I have been having that problem with several changesets the last week or so.

@joshix
Copy link
Contributor

joshix commented Dec 4, 2015

Needs:

  • address final review comments
  • branch conflict resolve
  • rebase to squash commits

then LGTM to land as an improvement iteration.

We can worry about unbreaking the lines later as we formalize coreos/docs#633 and continue to iterate on all of rkt/Documentation/.

One sentence per line. Add some whitespace between caption and the
text or between text and bullet listings. Remove some whitespace
errors. Fix language tag in some code snippets. Maybe fix some minor
typos.
rkt usually will not be a static binary. We tried once and the result
has problems with networking. It is probably like this because
statically linking glibc into a binary makes the glibc not to use its
plugin based name service switch fully, which is responsible, among
others, for resolving domain names.
Let's default to Carly Container with [email protected] email. Taken
from appc/spec docs.
@krnowak krnowak force-pushed the krnowak/docs-fixes branch from df995e9 to 94088a4 Compare December 7, 2015 09:53
Most of them spotted by Josh Wood
@krnowak krnowak force-pushed the krnowak/docs-fixes branch from 94088a4 to e8a5bea Compare December 7, 2015 09:56
@iaguis
Copy link
Member

iaguis commented Dec 7, 2015

LGTM

krnowak added a commit that referenced this pull request Dec 7, 2015
Documentation: fix formatting, typos, outdated info
@krnowak krnowak merged commit 4a67d68 into rkt:master Dec 7, 2015
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