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

Standardize operator docs #179

Closed btroncone closed 6 years ago

btroncone commented 6 years ago

This standardizes common discrepancies I found while going through the operator docs. All examples have been updated to use const, code references have been updated to the correct styling, and links have been corrected so they do not cause a page refresh.

I also removed rx-marbles for now as it was breaking if the page started on operators without an interactive marble diagram. Will need to research the cause of this issue.

codecov-io commented 6 years ago

Codecov Report

Merging #179 into master will decrease coverage by 10.92%. The diff coverage is 56.41%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #179       +/-   ##
===========================================
- Coverage   85.71%   74.78%   -10.93%     
===========================================
  Files           8        9        +1     
  Lines          91      119       +28     
  Branches        4        6        +2     
===========================================
+ Hits           78       89       +11     
- Misses         13       30       +17
Impacted Files Coverage Δ
src/app/core/services/seo.service.ts 70% <ø> (ø)
src/app/app.component.ts 57.14% <100%> (ø) :arrow_up:
src/app/material/material.module.ts 100% <100%> (ø)
src/app/core/core.module.ts 100% <100%> (ø)
...c/app/core/components/toolbar/toolbar.component.ts 85.71% <100%> (ø)
src/app/core/services/copier.service.ts 26.08% <26.08%> (ø)

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 f65c556...c0bc8a4. Read the comment docs.

ashwin-sureshkumar commented 6 years ago

@btroncone - please rebase