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

Add installation instruction in READEME.md #94

Merged
merged 1 commit into from
Sep 8, 2016

Conversation

jjchern
Copy link
Contributor

@jjchern jjchern commented May 7, 2016

No description provided.

@coveralls
Copy link

coveralls commented May 7, 2016

Coverage Status

Coverage increased (+0.6%) to 98.571% when pulling 03d36cc on jjchern:patch-1 into fa923de on Chicago:master.

@geneorama
Copy link
Member

@tomschenkjr I don't understand why the change to the readme would affect code coverage? Have you seen this before?

Also, I assume the Travis problem is not a real problem, but an artifact of the conflict created when the "insert" test twice by different computers?

Otherwise I don't see a problem with accepting this.

@tomschenkjr tomschenkjr changed the base branch from master to dev September 8, 2016 17:02
@tomschenkjr tomschenkjr merged commit 1e65927 into Chicago:dev Sep 8, 2016
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.

None yet

4 participants