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

Wrong function name in Either.filterOrElse JavaDoc #2618

Merged
merged 1 commit into from
Nov 29, 2020

Conversation

gualtierotesta
Copy link
Contributor

Small fix on Either.filterOrElse JavaDoc: it was referring to function filterVal while its name is zero

@codecov-io
Copy link

Codecov Report

Merging #2618 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2618   +/-   ##
=========================================
  Coverage     92.87%   92.87%           
  Complexity     5360     5360           
=========================================
  Files            89       89           
  Lines         12777    12777           
  Branches       1619     1619           
=========================================
  Hits          11866    11866           
  Misses          722      722           
  Partials        189      189           
Impacted Files Coverage Δ Complexity Δ
src/main/java/io/vavr/control/Either.java 97.34% <ø> (ø) 59.00 <0.00> (ø)

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 d6f9ccb...03d3138. Read the comment docs.

@gualtierotesta gualtierotesta changed the title Wrong parameter name in Either.filterOrElse JavaDoc Wrong function name in Either.filterOrElse JavaDoc Oct 25, 2020
@danieldietrich danieldietrich modified the milestone: v1.0.0 Nov 29, 2020
@danieldietrich
Copy link
Contributor

Thank you!

@danieldietrich danieldietrich merged commit 52d4591 into vavr-io:master Nov 29, 2020
danieldietrich pushed a commit that referenced this pull request Jul 14, 2021
Co-authored-by: Gualtiero Testa <gualtiero.testa@pm.me>
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