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

2.x: fixes, cleanups, renames #4421

Merged
merged 2 commits into from
Aug 24, 2016
Merged

Conversation

akarnokd
Copy link
Member

@akarnokd akarnokd commented Aug 24, 2016

Changes:

  • fix spelling mistakes
  • more consistent parameter naming scheme based on Make operator parameter names consistent across operators. #1353 (but incomplete, RC2)
  • fix bugs due to wrong variable naming
  • compact logic expressions
  • fix Completable.doAfterTerminate not called the right time
  • onErrorReturnValue renamed to onErrorReturnItem

@DavidMGross once this PR has been merged, it would be great you submitted one that has //TODO markers on those lines that need attention. The naming scheme you suggested doesn't come intuitively to me so I may skip many of them without the markers.

@codecov-io
Copy link

Current coverage is 69.48% (diff: 73.27%)

Merging #4421 into 2.x will increase coverage by 0.03%

@@                2.x      #4421   diff @@
==========================================
  Files           454        454          
  Lines         32372      32323    -49   
  Methods           0          0          
  Messages          0          0          
  Branches       5214       5214          
==========================================
- Hits          22482      22459    -23   
+ Misses         7692       7662    -30   
- Partials       2198       2202     +4   

Powered by Codecov. Last update 35c8da6...b05aa63

@JakeWharton
Copy link
Contributor

👍

@akarnokd akarnokd merged commit 7aa0b34 into ReactiveX:2.x Aug 24, 2016
@akarnokd akarnokd deleted the CleanupIntelliJ branch August 24, 2016 20:02
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