Add toNev to NonEmptyList variants #3423
Conversation
| *}}} | ||
| */ | ||
| final def toNev[B >: A]: NonEmptyVector[B] = | ||
| NonEmptyVector.fromVectorUnsafe(toLazyList.toList.toVector) |
There was a problem hiding this comment.
does LazyList not have its own toVector method?
There was a problem hiding this comment.
You are right. It does. Additionally, I was missing an import. I will push changes. Thank you @LukaJCB :)
Codecov Report
@@ Coverage Diff @@
## master #3423 +/- ##
=========================================
Coverage ? 92.71%
=========================================
Files ? 379
Lines ? 7988
Branches ? 229
=========================================
Hits ? 7406
Misses ? 582
Partials ? 0
Continue to review full report at Codecov.
|
| * res0: Boolean = true | ||
| *}}} | ||
| */ | ||
| def toNev[B >: A]: NonEmptyVector[B] = |
There was a problem hiding this comment.
Is there a reason this one isn't final but the one above is?
There was a problem hiding this comment.
Yes. NonEmptyList is a final class. LazyNonEmptyList is not.
Closes #3422