Closed JozefFlakus closed 1 year ago
Merging #400 (02d9374) into master (cbdffda) will decrease coverage by
1.08%
. The diff coverage is93.19%
.
@@ Coverage Diff @@
## master #400 +/- ##
==========================================
- Coverage 95.41% 94.32% -1.09%
==========================================
Files 157 158 +1
Lines 2769 3103 +334
Branches 366 358 -8
==========================================
+ Hits 2642 2927 +285
- Misses 123 172 +49
Partials 4 4
Impacted Files | Coverage Δ | |
---|---|---|
packages/core/src/logger/logger.interface.ts | 100.00% <ø> (ø) |
|
packages/core/src/operators/index.ts | 100.00% <ø> (ø) |
|
packages/core/src/operators/use/use.operator.ts | 50.00% <ø> (-50.00%) |
:arrow_down: |
packages/http/src/http.interface.ts | 100.00% <ø> (ø) |
|
packages/http/src/index.ts | 0.00% <0.00%> (ø) |
|
packages/http/src/router/http.router.combiner.ts | 92.30% <ø> (ø) |
|
packages/http/src/router/http.router.matcher.ts | 100.00% <ø> (ø) |
|
...ages/http/src/router/http.router.params.factory.ts | 100.00% <ø> (ø) |
|
packages/http/src/server/http.server.interface.ts | 100.00% <ø> (ø) |
|
packages/messaging/src/reply/reply.ts | 100.00% <ø> (ø) |
|
... and 126 more |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
PR Type
What kind of change does this PR introduce?
What is the current behavior?
When calling
ackMessage
/nackMesage
in case of redis strategy the listener returns an errored event together with a respective error log.What is the new behavior?
rxjs
,fp-ts
,io-ts
to the latest supported version (closes: #396)^7.5.7
^2.13.1
^2.2.19
ackMessage
/nackMesage
in case of redis strategy the listener doesn't return an errored event, instead only a respective error log, so that the related effect can run without any disturbanceDoes this PR introduce a breaking change?