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 a README explaining the backpressure #84

Merged
merged 2 commits into from
Feb 16, 2019

Conversation

backuitist
Copy link
Contributor

It's not complete but perhaps a good starting point :)

@backuitist backuitist force-pushed the readme-backpressure branch 3 times, most recently from afedc3f to d581ba0 Compare February 12, 2019 09:39
@codecov
Copy link

codecov bot commented Feb 12, 2019

Codecov Report

Merging #84 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   98.34%   98.34%           
=======================================
  Files          34       34           
  Lines         725      725           
  Branches       51       51           
=======================================
  Hits          713      713           
  Misses         12       12

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf301e9...97f1f17. Read the comment docs.

@backuitist
Copy link
Contributor Author

I thought that the site would contain user focused documentation, whereas the README would contain instructions for contributors. I like when you don't have to click around to find the information :)
Feel free to move this to the documentation if you disagree though :)

@vlovgr
Copy link
Contributor

vlovgr commented Feb 12, 2019

Thanks a lot for this, @backuitist! I believe it's nice to have all high-level documentation in one place. I'll take a stab at moving this to the site and adding some notes from myself as well, when I get some time.

@vlovgr vlovgr merged commit 583ca4a into fd4s:master Feb 16, 2019
@backuitist
Copy link
Contributor Author

Neat! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants