Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix readme for TokTok #12

Merged
merged 3 commits into from
Jul 12, 2016
Merged

Fix readme for TokTok #12

merged 3 commits into from
Jul 12, 2016

Conversation

GrayHatter
Copy link

@GrayHatter GrayHatter commented Jul 11, 2016

@iphydf


This change is Reviewable

@iphydf
Copy link
Member

iphydf commented Jul 11, 2016

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.


README.md, line 4 [r1] (raw file):

***

**Current build status:** [![Build Status](https://travis-ci.org/TokTok/toxcore.svg?branch=master)](https://travis-ci.org/TokTok/toxcore)

Can you also add the coverage status? See https://raw.githubusercontent.com/TokTok/hstox/master/README.md for an example.


Comments from Reviewable

@iphydf
Copy link
Member

iphydf commented Jul 11, 2016

Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions.


README.md, line 1 [r1] (raw file):

![Project Tox](https://raw.github.com/irungentoo/toxcore/master/other/tox.png "Project Tox")

This could also be changed, in case someone decides to contribute a different logo.


Comments from Reviewable

@GrayHatter
Copy link
Author

Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions.


README.md, line 1 [r1] (raw file):

Previously, iphydf wrote…

This could also be changed, in case someone decides to contribute a different logo.

Done.

README.md, line 4 [r1] (raw file):

Previously, iphydf wrote…

Can you also add the coverage status? See https://raw.githubusercontent.com/TokTok/hstox/master/README.md for an example.

Done.

Comments from Reviewable

@nurupo
Copy link
Member

nurupo commented Jul 12, 2016

LGTM, unless you want to fix IRC and other links.

@iphydf
Copy link
Member

iphydf commented Jul 12, 2016

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.


README.md, line 5 [r2] (raw file):

**Current build status:** [![Build Status](https://travis-ci.org/TokTok/toxcore.svg?branch=master)](https://travis-ci.org/TokTok/toxcore)
**Current Coverage :** [![Coverage Status](https://coveralls.io/repos/github/TokTok/toxcore/badge.svg?branch=master)](https://coveralls.io/github/TokTok/toxcore?branch=master)

Remove the extra space before ':'


Comments from Reviewable

Requested, suggested
@iphydf
Copy link
Member

iphydf commented Jul 12, 2016

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

This pull request was closed.
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.

3 participants