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

Builders as functions #12

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Builders as functions #12

wants to merge 4 commits into from

Conversation

lostintime
Copy link
Collaborator

ObservableInstance is now Observable
Old Observble static methods moved to separate module as functions

It can be now used as:

import * as Mx from "monix-reactive"

const stream: Mx.Observable<number> = Mx.pure(10).pipe(Mx.id)

May resolve #10

@codecov
Copy link

codecov bot commented Mar 6, 2018

Codecov Report

Merging #12 into master will decrease coverage by 0.29%.
The diff coverage is 94.44%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #12     +/-   ##
=========================================
- Coverage   89.51%   89.21%   -0.3%     
=========================================
  Files          15       14      -1     
  Lines         267      269      +2     
  Branches       36       36             
=========================================
+ Hits          239      240      +1     
- Misses         28       29      +1
Impacted Files Coverage Δ
...es/monix-reactive/src/internal/subscribers/safe.ts 71.15% <ø> (ø) ⬆️
...es/monix-reactive/src/internal/subscribers/wrap.ts 94.44% <ø> (ø) ⬆️
packages/monix-reactive/src/internal/ack-utils.ts 94.11% <ø> (ø) ⬆️
packages/monix-reactive/src/builders.ts 100% <100%> (ø)
...ages/monix-reactive/src/internal/builders/never.ts 100% <100%> (ø) ⬆️
...kages/monix-reactive/src/internal/builders/eval.ts 92.1% <100%> (ø) ⬆️
...kages/monix-reactive/src/internal/builders/pure.ts 100% <100%> (ø) ⬆️
...ages/monix-reactive/src/internal/builders/empty.ts 100% <100%> (ø) ⬆️
...ages/monix-reactive/src/internal/builders/array.ts 88.57% <100%> (ø) ⬆️
packages/monix-reactive/src/index.ts 100% <100%> (ø) ⬆️
... and 3 more

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 a9260c7...fc56cba. Read the comment docs.

@lostintime lostintime requested a review from alexandru March 12, 2018 06:00
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.

Observable class vs builders
1 participant