ReactiveX / rxjs-docs

The home for new work on the new RxJS docs (RxJS v 5 and up). New to this space? Say hi here: https://github.com/ReactiveX/rxjs-docs/issues/24. Want to find out what's up? We're chatting here. https://github.com/ReactiveX/rxjs-docs/issues/4
Apache License 2.0
160 stars 62 forks source link

docs(operators): add documentation for sample & sampleTime #207

Open hardikpthv opened 6 years ago

hardikpthv commented 6 years ago

Closes: #192

codecov-io commented 6 years ago

Codecov Report

Merging #207 into master will increase coverage by 0.04%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
+ Coverage   90.78%   90.82%   +0.04%     
==========================================
  Files         116      117       +1     
  Lines         445      447       +2     
  Branches        9        9              
==========================================
+ Hits          404      406       +2     
  Misses         40       40              
  Partials        1        1
Impacted Files Coverage Δ
src/operator-docs/filtering/sample.ts 100% <ø> (ø) :arrow_up:
src/operator-docs/filtering/index.ts 100% <100%> (ø) :arrow_up:
src/operator-docs/filtering/sampleTime.ts 100% <100%> (ø)

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 4097607...82e67e4. Read the comment docs.

ashwin-sureshkumar commented 6 years ago

@hardikpthv - awesome job !!

Please make sure there is a single operator per PR. Easier to organize and track.

Also there is already a PR for sample. I will close it. Let's make sure we don't repeat work :)

hardikpthv commented 6 years ago

Oh i see! @ashwin-sureshkumar I will keep it in mind. :)

ashwin-sureshkumar commented 6 years ago

@sumitarora @btroncone - Can you please re-review this PR?

ashwin-sureshkumar commented 6 years ago

@btroncone @sumitarora - please re-review when get a chance

ashwin-sureshkumar commented 6 years ago

@btroncone @sumitarora - please re-review when get a chance

ashwin-sureshkumar commented 6 years ago

@btroncone @sumitarora - please re-review when get a chance

hardikpthv commented 6 years ago

I think sample operator can be ignored as it's already there.

hardikpthv commented 6 years ago

can this be merged @ashwin-sureshkumar or any changes expected?