Skip to content
This repository was archived by the owner on Oct 1, 2018. It is now read-only.

Conversation

@ashwin-sureshkumar
Copy link
Collaborator

Close #101

@codecov-io
Copy link

codecov-io commented Dec 7, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #186   +/-   ##
=======================================
  Coverage   74.78%   74.78%           
=======================================
  Files           9        9           
  Lines         119      119           
  Branches        6        6           
=======================================
  Hits           89       89           
  Misses         30       30

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 8804ed5...3ec2696. Read the comment docs.

},
walkthrough: {
description: `<p>
<code>take</code> returns an Observable that emits only the first count values emitted by the source Observable.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Instead of <code> can we use the markdown class for consistent styling?

@ashwin-sureshkumar
Copy link
Collaborator Author

@btroncone - done !

@btroncone
Copy link
Collaborator

lgtm 👍

@btroncone btroncone merged commit e7bbcb2 into ReactiveX:master Dec 8, 2017
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.

3 participants