search

frintjs / frint

Modular JavaScript framework for building scalable and reactive applications
https://frint.js.org/
MIT License
756 stars 37 forks source link

frint-cli: fix #321

Closed fahad19 closed 7 years ago

fahad19 commented 7 years ago

Problem

Since we now import RxJS module individually, the operators have to be imported manually too.

The bin for frint-cli was using operators in a chained manner. And didn't run because of that.

Solution

Since frint-cli is written without involving any transpiler, I am wrapping the apps$ stream to expose all operators as methods.

Ideal fix in future

Get a src / lib based workflow like all other packages, and not load full RxJS for CLI usage.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 97.277% when pulling b32ee8b41e285e98802bf472fc49a6fa19d8adce on cli-fix into 5092bc275c45e83d5965037f6c53f7fece51986e on master.