Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

docs: Add link to install guides in README #538

Merged

Conversation

jodh-intel
Copy link
Contributor

Although the wiki includes links to the installation guides,
so should the README.

Fixes #537.

Signed-off-by: James O. D. Hunt [email protected]

@jodh-intel
Copy link
Contributor Author

Note the link to the docs directory. We could create a nicely-formatted list of installation guides in the README, but that's more maintenance. We could also have the README link to the wiki, but since the wiki links back to the repo, that seems a bit odd circuitous ;)

@coveralls
Copy link

coveralls commented Sep 11, 2017

Coverage Status

Coverage remained the same at 97.04% when pulling 200faf7 on jodh-intel:add-link-to-install-docs into 1fe09bb on clearcontainers:master.

@clearcontainersbot
Copy link

Popular Images qa-passed 👍

@jodh-intel
Copy link
Contributor Author

Hi @iphutch - please can you review?

README.md Outdated
@@ -33,6 +33,10 @@ The runtime has a built-in command to determine if your host system is capable o
$ cc-runtime cc-check
```

## Quick start for users

See the installation guides available for various operating systems [here](docs/).
Copy link

Choose a reason for hiding this comment

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

So that we can avoid "here", lets move the link to "installation guides" as has been done in the two sections following this one:
See the installation guides available for various operating systems.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@coveralls
Copy link

coveralls commented Sep 13, 2017

Coverage Status

Coverage remained the same at 97.04% when pulling 71c4a11 on jodh-intel:add-link-to-install-docs into fa07228 on clearcontainers:master.

@clearcontainersbot
Copy link

Popular Images qa-passed 👍

@jodh-intel
Copy link
Contributor Author

@iphutch and approvers - please can you review?

@coveralls
Copy link

coveralls commented Sep 14, 2017

Coverage Status

Coverage remained the same at 97.052% when pulling 35acce1 on jodh-intel:add-link-to-install-docs into 97aa6e6 on clearcontainers:master.

@clearcontainersbot
Copy link

Popular Images qa-passed 👍

@coveralls
Copy link

coveralls commented Sep 14, 2017

Coverage Status

Coverage remained the same at 97.052% when pulling 840ebb7 on jodh-intel:add-link-to-install-docs into 97aa6e6 on clearcontainers:master.

@clearcontainersbot
Copy link

Popular Images qa-passed 👍

Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

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

lgtm

@coveralls
Copy link

coveralls commented Sep 14, 2017

Coverage Status

Coverage remained the same at 96.825% when pulling 5267182 on jodh-intel:add-link-to-install-docs into 91f2b3e on clearcontainers:master.

Copy link

@iphutch iphutch left a comment

Choose a reason for hiding this comment

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

lgtm

Although the wiki includes links to the installation guides,
so should the README.

Fixes clearcontainers#537.

Signed-off-by: James O. D. Hunt <[email protected]>
@coveralls
Copy link

coveralls commented Sep 15, 2017

Coverage Status

Coverage remained the same at 96.825% when pulling b0e8112 on jodh-intel:add-link-to-install-docs into 0f8ec51 on clearcontainers:master.

@jodh-intel jodh-intel merged commit 038174e into clearcontainers:master Sep 15, 2017
mcastelino pushed a commit to mcastelino/runtime that referenced this pull request Dec 6, 2018
So that shared memory guests can still generate different
random numbers.

Fixes: clearcontainers#538

Signed-off-by: Peng Tao <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants