Skip to content

Add README and some lib.rs docs#31

Merged
Hoverbear merged 3 commits intomasterfrom
readme-and-lib-docs
Feb 15, 2019
Merged

Add README and some lib.rs docs#31
Hoverbear merged 3 commits intomasterfrom
readme-and-lib-docs

Conversation

@Hoverbear
Copy link
Copy Markdown
Contributor

@Hoverbear Hoverbear commented Feb 12, 2019

This adds a first pass of README and the crate level docs.

Edit (brson): Rendered README

@Hoverbear Hoverbear self-assigned this Feb 12, 2019
@Hoverbear Hoverbear requested review from QueenyJin and brson February 12, 2019 00:50
@QueenyJin
Copy link
Copy Markdown

@dcalvin PTAL.

@QueenyJin QueenyJin requested a review from dcalvin February 12, 2019 02:18
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread src/transaction.rs Outdated
Comment thread src/raw.rs Outdated
Comment thread src/raw.rs Outdated
Comment thread src/lib.rs Outdated
Comment thread src/lib.rs Outdated
Comment thread README.md Outdated
Comment thread src/lib.rs Outdated
@Hoverbear Hoverbear force-pushed the readme-and-lib-docs branch 2 times, most recently from 750c7a6 to 43fa973 Compare February 12, 2019 19:13
Copy link
Copy Markdown
Contributor

@brson brson left a comment

Choose a reason for hiding this comment

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

Looks good. Please answer my questions and resolve my comments as you see fit, and I'll approve.

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread src/lib.rs Outdated
Comment thread README.md Outdated
Comment thread src/lib.rs Outdated
Comment thread src/lib.rs
Comment thread src/lib.rs
Comment thread src/lib.rs Outdated
Comment thread src/lib.rs Outdated
@Hoverbear Hoverbear removed the request for review from QueenyJin February 13, 2019 21:13
Copy link
Copy Markdown
Member

@dcalvin dcalvin left a comment

Choose a reason for hiding this comment

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

LGTM

@Hoverbear Hoverbear requested a review from brson February 14, 2019 17:20
@Hoverbear Hoverbear mentioned this pull request Feb 14, 2019
3 tasks
@brson
Copy link
Copy Markdown
Contributor

brson commented Feb 15, 2019

Needs sign off, but it's good to merge after.

Signed-off-by: Ana Hobden <operator@hoverbear.org>
Nit fixes.

Signed-off-by: Ana Hobden <operator@hoverbear.org>
Signed-off-by: Ana Hobden <operator@hoverbear.org>
@Hoverbear
Copy link
Copy Markdown
Contributor Author

Merging, CI still red.

@Hoverbear Hoverbear merged commit a5a6b15 into master Feb 15, 2019
@ice1000
Copy link
Copy Markdown

ice1000 commented Feb 15, 2019

I believe CI is failing due to the organization transfer.

@Hoverbear
Copy link
Copy Markdown
Contributor Author

@ice1000 The CI is failing because of the recent kvproto update as well as the need for a recent raft release. :(

pingcap/kvproto#354
pingcap/kvproto#355
tikv/raft-rs#182

@ice1000
Copy link
Copy Markdown

ice1000 commented Feb 15, 2019

Wooo. So the badge's URL in README need to be updated.

@sticnarf sticnarf deleted the readme-and-lib-docs branch April 7, 2020 03:56
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.

5 participants