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

Fix: CharSeq implements Comparable<CharSeq> #2399

Merged
merged 4 commits into from
Mar 31, 2019

Conversation

thadumi
Copy link
Contributor

@thadumi thadumi commented Mar 27, 2019

Fixed the bug #2382

Theodor A. Dumitrescu added 3 commits March 27, 2019 13:23
class io.vavr.collection.CharSeq cannot be cast to
class java.lang.Comparable
class io.vavr.collection.CharSeq cannot be cast to
class java.lang.Comparable
@codecov-io
Copy link

codecov-io commented Mar 27, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2399   +/-   ##
=========================================
  Coverage     95.79%   95.79%           
  Complexity     5427     5427           
=========================================
  Files            94       94           
  Lines         13110    13110           
  Branches       1608     1608           
=========================================
  Hits          12559    12559           
  Misses          404      404           
  Partials        147      147
Impacted Files Coverage Δ Complexity Δ
vavr/src/main/java/io/vavr/collection/CharSeq.java 99.82% <ø> (ø) 349 <0> (ø) ⬇️

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 f34c252...fc05372. Read the comment docs.

Copy link
Contributor

@danieldietrich danieldietrich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Theodor!
Many thanks :-)
Please find my review comments.
Greets,
Daniel

…arSeq

as java.lang.Comparator.
This commit concerns the issue vavr-io#2382.
@danieldietrich danieldietrich modified the milestones: v0.11.0, v0.10.1 Mar 31, 2019
@danieldietrich
Copy link
Contributor

Many thanks!! :)

@danieldietrich danieldietrich merged commit 8344e95 into vavr-io:master Mar 31, 2019
@danieldietrich danieldietrich modified the milestones: v0.10.1, v0.11.0 Jul 2, 2019
@danieldietrich danieldietrich changed the title [Fix][Bug] #2382 CharSeq implements Comparable<CharSeq> Jul 8, 2019
@danieldietrich danieldietrich changed the title CharSeq implements Comparable<CharSeq> Fix: CharSeq implements Comparable<CharSeq> Jul 8, 2019
@danieldietrich danieldietrich modified the milestones: v1.0.0, v0.10.1 Jul 8, 2019
danieldietrich pushed a commit that referenced this pull request Jul 22, 2019
* [Fix][Bug][2382] java.lang.ClassCastException:
class io.vavr.collection.CharSeq cannot be cast to
class java.lang.Comparable

* [Fix][Bug][2382] java.lang.ClassCastException:
class io.vavr.collection.CharSeq cannot be cast to
class java.lang.Comparable

* Added a test case for the bug fix #2382.

* Added less, greater and less test case for testing the behavior of CharSeq
as java.lang.Comparator.
This commit concerns the issue #2382.
thadumi pushed a commit to thadumi/vavr that referenced this pull request Dec 11, 2019
* [Fix][Bug][2382] java.lang.ClassCastException:
class io.vavr.collection.CharSeq cannot be cast to
class java.lang.Comparable

* [Fix][Bug][2382] java.lang.ClassCastException:
class io.vavr.collection.CharSeq cannot be cast to
class java.lang.Comparable

* Added a test case for the bug fix vavr-io#2382.

* Added less, greater and less test case for testing the behavior of CharSeq
as java.lang.Comparator.
This commit concerns the issue vavr-io#2382.
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