Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify Uni to Multi bridge #631

Merged
merged 1 commit into from
Jul 20, 2021
Merged

Simplify Uni to Multi bridge #631

merged 1 commit into from
Jul 20, 2021

Conversation

jponge
Copy link
Member

@jponge jponge commented Jul 20, 2021

No description provided.

@jponge jponge added this to the 1.0.0 milestone Jul 20, 2021
@jponge jponge added enhancement New feature or request technical-debt Technical debt reduction labels Jul 20, 2021
@jponge jponge requested a review from cescoffier July 20, 2021 10:25
@jponge
Copy link
Member Author

jponge commented Jul 20, 2021

This shortcut saves allocations

@codecov
Copy link

codecov bot commented Jul 20, 2021

Codecov Report

Merging #631 (b4b19bf) into main (941c555) will increase coverage by 0.31%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #631      +/-   ##
============================================
+ Coverage     89.93%   90.24%   +0.31%     
- Complexity     3015     3024       +9     
============================================
  Files           395      395              
  Lines         11829    11829              
  Branches       1478     1480       +2     
============================================
+ Hits          10638    10675      +37     
+ Misses          607      590      -17     
+ Partials        584      564      -20     
Impacted Files Coverage Δ
...rye/mutiny/converters/uni/UniToMultiPublisher.java 88.88% <ø> (ø)
...java/io/smallrye/mutiny/operators/AbstractUni.java 90.00% <100.00%> (ø)
...mallrye/mutiny/helpers/queues/MpscLinkedQueue.java 90.32% <0.00%> (-8.07%) ⬇️
...mallrye/mutiny/operators/multi/MultiFlatMapOp.java 85.04% <0.00%> (+0.62%) ⬆️
...smallrye/mutiny/operators/multi/MultiEmitOnOp.java 90.47% <0.00%> (+0.95%) ⬆️
...allrye/mutiny/operators/uni/UniAndCombination.java 89.53% <0.00%> (+1.16%) ⬆️
...erators/multi/builders/SerializedMultiEmitter.java 82.27% <0.00%> (+1.26%) ⬆️
...tiny/operators/multi/MultiBufferWithTimeoutOp.java 74.78% <0.00%> (+1.68%) ⬆️
...subscription/SwitchableSubscriptionSubscriber.java 97.54% <0.00%> (+2.45%) ⬆️
...y/operators/multi/builders/IterableBasedMulti.java 86.17% <0.00%> (+4.25%) ⬆️
... and 5 more

@cescoffier cescoffier merged commit 165a9c9 into main Jul 20, 2021
@cescoffier cescoffier deleted the fix/simpler-uni-to-multi branch July 20, 2021 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request technical-debt Technical debt reduction
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants