Skip to content

1.x: fix SingleTest generics warnings #4254

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

Merged
merged 1 commit into from
Jul 28, 2016

Conversation

akarnokd
Copy link
Member

This fixes the generics warnings in the new Single.flatMapCompletable unit tests.

@vanniktech
Copy link
Collaborator

👍 didn't see those since I have Java 8 as default in my IDE

@akarnokd
Copy link
Member Author

I'm using Eclipse which turned out to be an excellent tool to warn about these things. Are you using IntelliJ? I've noticed that people using IntelliJ end up with such hidden problems (while IntelliJ for me complained about almost everything else when I loaded RxJava into it once).

@codecov-io
Copy link

codecov-io commented Jul 28, 2016

Current coverage is 84.30% (diff: 100%)

Merging #4254 into 1.x will decrease coverage by 0.04%

@@                1.x      #4254   diff @@
==========================================
  Files           266        266          
  Lines         17446      17446          
  Methods           0          0          
  Messages          0          0          
  Branches       2657       2657          
==========================================
- Hits          14715      14707     -8   
- Misses         1870       1879     +9   
+ Partials        861        860     -1   

Powered by Codecov. Last update 41959f7...acfdde2

@akarnokd akarnokd merged commit 888560e into ReactiveX:1.x Jul 28, 2016
@akarnokd akarnokd deleted the TestCleanup728 branch July 28, 2016 14:23
@vanniktech
Copy link
Collaborator

Yes I'm using Android Studio which is based on IntelliJ. That's weird I had no problems loading the project into AS.

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

Successfully merging this pull request may close these issues.

3 participants