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

Update Backpressure.md #6685

Merged
merged 1 commit into from
Nov 1, 2019
Merged

Update Backpressure.md #6685

merged 1 commit into from
Nov 1, 2019

Conversation

maryang710
Copy link

Fixes Backpressure.md on #6132
Fix Images on Backpressure.md crashed on wrong address. Put full address of images.


Thank you for contributing to RxJava. Before pressing the "Create Pull Request" button, please consider the following points:

  • [v] Please give a description about what and why you are contributing, even if it's trivial.

  • [v] Please include the issue list number(s) or other PR numbers in the description if you are contributing in response to those.

  • Please include a reasonable set of unit tests if you contribute new code or change an existing one. If you contribute an operator, (if applicable) please make sure you have tests for working with an empty, just, range of values as well as an error source, with and/or without backpressure and see if unsubscription/cancellation propagates correctly.

Fixes Backpressure.md on ReactiveX#6132 

Fix Images on Backpressure.md crashed on wrong address. Put full address of images.
@codecov
Copy link

codecov bot commented Oct 24, 2019

Codecov Report

Merging #6685 into 2.x will increase coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #6685      +/-   ##
============================================
+ Coverage     98.15%   98.22%   +0.06%     
- Complexity     6345     6349       +4     
============================================
  Files           677      677              
  Lines         45550    45550              
  Branches       6333     6333              
============================================
+ Hits          44711    44740      +29     
+ Misses          267      254      -13     
+ Partials        572      556      -16
Impacted Files Coverage Δ Complexity Δ
...rnal/operators/flowable/FlowableFlatMapSingle.java 92.93% <0%> (-2.72%) 2% <0%> (ø)
...ernal/operators/flowable/FlowableFlatMapMaybe.java 90.82% <0%> (-0.97%) 2% <0%> (ø)
...perators/single/SingleFlatMapIterableFlowable.java 96.66% <0%> (-0.84%) 2% <0%> (ø)
...l/operators/observable/ObservableFlatMapMaybe.java 85.62% <0%> (-0.66%) 2% <0%> (ø)
...ernal/operators/flowable/FlowableFromIterable.java 96.25% <0%> (-0.54%) 5% <0%> (ø)
...ternal/operators/flowable/FlowableWindowTimed.java 91.4% <0%> (-0.46%) 4% <0%> (ø)
...al/operators/observable/ObservableWindowTimed.java 90.73% <0%> (+0.27%) 4% <0%> (ø) ⬇️
...ex/internal/operators/flowable/FlowableCreate.java 97.41% <0%> (+0.32%) 6% <0%> (ø) ⬇️
...ternal/operators/observable/ObservableFlatMap.java 88.99% <0%> (+0.32%) 3% <0%> (ø) ⬇️
...internal/operators/flowable/FlowableSwitchMap.java 95.37% <0%> (+0.46%) 3% <0%> (ø) ⬇️
... and 13 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 90ae2a3...7901b69. Read the comment docs.

@munayam81
Copy link

Raaz i

@akarnokd akarnokd merged commit d0a59e1 into ReactiveX:2.x Nov 1, 2019
@akarnokd
Copy link
Member

akarnokd commented Nov 1, 2019

Thanks for the contribution. Please next time target the 3.x branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants