marblejs / marble

Marble.js - functional reactive Node.js framework for building server-side applications, based on TypeScript and RxJS.
https://marblejs.com
MIT License
2.15k stars 69 forks source link

fix(websockets): added `path-to-regexp` package to the dependency list #348

Closed JozefFlakus closed 2 years ago

JozefFlakus commented 2 years ago

PR Type

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #346

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No
codecov[bot] commented 2 years ago

Codecov Report

Merging #348 (50c2013) into master (cbdffda) will increase coverage by 0.31%. The diff coverage is 96.41%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #348      +/-   ##
==========================================
+ Coverage   95.41%   95.72%   +0.31%     
==========================================
  Files         157      163       +6     
  Lines        2769     3114     +345     
  Branches      366      362       -4     
==========================================
+ Hits         2642     2981     +339     
- Misses        123      128       +5     
- Partials        4        5       +1     
Impacted Files Coverage Δ
packages/core/src/context/context.hook.ts 100.00% <ø> (ø)
...ackages/core/src/effects/effectsContext.factory.ts 100.00% <ø> (ø)
packages/core/src/logger/logger.interface.ts 100.00% <ø> (ø)
packages/messaging/src/reply/reply.ts 100.00% <ø> (ø)
...rc/transport/strategies/amqp.strategy.interface.ts 81.81% <ø> (ø)
.../testing/src/testBed/http/http.testBed.response.ts 75.00% <40.00%> (-13.24%) :arrow_down:
packages/core/src/+internal/utils/string.util.ts 92.00% <66.66%> (-8.00%) :arrow_down:
packages/core/src/+internal/utils/any.util.ts 84.61% <71.42%> (+1.28%) :arrow_up:
...ore/src/http/response/http.responseBody.factory.ts 80.95% <73.33%> (-19.05%) :arrow_down:
...ssaging/src/transport/strategies/local.strategy.ts 92.15% <83.33%> (+4.92%) :arrow_up:
... and 72 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 0997bdf...50c2013. Read the comment docs.