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
CU-fb6x32 Clean up Arrow Syntax based on API Review #2330
Conversation
nomisRev
commented
Mar 17, 2021
•
edited
edited
- Remove Ior.unit
- Remove Validated.s
- Remove SetKZip instance
- Remove foldMapA
- Remove ifM
- Remove zipEval
- Remove selectM
- Remove fproduct/tupleLeft/tupleRight/mproduct
- ap/apEval
- Deprecate tailRecM
- Update deprecations AndThen
Task linked: CU-fv6x32 Clean-up Arrow Core after API Review |
arrow-libs/core/arrow-core-data/src/test/kotlin/arrow/core/IterableTest.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core-data/src/test/kotlin/arrow/core/SequenceKTest.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nomisRev looking great! thanks 🙏
arrow-libs/core/arrow-core-data/src/main/kotlin/arrow/core/AndThen.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core-data/src/main/kotlin/arrow/core/Nullable.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core-data/src/main/kotlin/arrow/core/predef.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core-data/src/test/kotlin/arrow/core/IterableTest.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/ior/monad/IorMonad.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/list/apply/ListKApply.kt
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/list/apply/ListKApply.kt
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/list/monad/ListKMonad.kt
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/option/apply/OptionApply.kt
Show resolved
Hide resolved
Co-authored-by: Raúl Raja Martínez <raulraja@gmail.com>
…ns/ior/apply/IorApply.kt Co-authored-by: Raúl Raja Martínez <raulraja@gmail.com>
…ns/ior/monad/IorMonad.kt Co-authored-by: Raúl Raja Martínez <raulraja@gmail.com>
…ns/ior/monad/IorMonad.kt Co-authored-by: Raúl Raja Martínez <raulraja@gmail.com>
…ns/ior/monad/IorMonad.kt Co-authored-by: Raúl Raja Martínez <raulraja@gmail.com>
…ow-kt/arrow into cu-fv6x32-clean-up-arrow-core-review
arrow-libs/core/arrow-core-data/src/main/kotlin/arrow/core/Const.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core-data/src/main/kotlin/arrow/core/Ior.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/either/monad/EitherMonad.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/either/monad/EitherMonad.kt
Outdated
Show resolved
Hide resolved
arrow-libs/core/arrow-core/src/main/kotlin/arrow/core/extensions/eval/apply/EvalApply.kt
Outdated
Show resolved
Hide resolved
…ns/either/monad/EitherMonad.kt
…ns/eval/apply/EvalApply.kt
…ns/either/monad/EitherMonad.kt
…ow-kt/arrow into cu-fv6x32-clean-up-arrow-core-review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @nomisRev 🙏
inline fun <E, A, B> Iterable<A>.traverseEither(f: (A) -> Either<E, B>): Either<E, List<B>> = | ||
foldRight<A, Either<E, List<B>>>(emptyList<B>().right()) { a, acc -> | ||
f(a).ap(acc.map { bs -> { b: B -> listOf(b) + bs } }) | ||
when (val res = f(a)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙌🏾
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
my longest review of the week 😄
Thanks for the work @nomisRev 🙌🏾