Sergey Torgashov
Sergey Torgashov
Well, for Cats it's definitely disabled because when I enable it temporarily and re-compile for 2.12, then the "unused" warnings begin coming in numbers.
@armanbilge Initially I intended to fix compiler warnings in a bulk. And I completed with "kernel" modules quite a long ago. But turned out that "core" modules have quite a...
Ok nice, thank you. I think this PR is pretty much ready then.
@armanbilge yeah, no worries. I also feel a bit hesitant about pushing this work out. Because before continuing it would be better to stabilize the build system itself. > However...
Converted to "Draft" to put on hold until work on scalac compiler options is stabilized.
I agree that such a method could be a really nice addition. A couple of thoughts though: 1. Name `groupByNelMap` or `groupByMapNel`? IMO the latter seems to be a more...
> I've chosen `groupByNelMap` because the underlying method `groupByMap` name is constructed like `groupBy` + `map`. In that sense, `groupByNelMap` is `groupByNel` (that exists in syntax) + `map`. Actually, there...
> I agree with adding that method to other syntaxes, but I feel we can do it iteratively? Yes sure, totally up to you. I mean, it would be really...
> @satorg Thanks for your thoughts on this, but `groupMapNel` is kinda counterintuitive for me. Naming is controversial all the time, so it would be nice to get more opinions....
Let me enumerate all group-like methods we have both in Scala library and Cats (just for everyone's convenience). I didn't include LazyList/NonEmptyLazyList and Stream/NonEmptyStream. | Scala | source | target...