r
Untitled.txt
a
i’m only missing the arrow-fx-kotlinx-coroutines in arrow-fx
r
Right, it was recently added in Arrow. Thanks @aballano!!
👍 1
s
I think we should move
arrow-fx-mtl
to
arrow-mtl
. Sorry to bring this up so late, but I just considered that we’re trying running into complex deprecation cycle in
arrow-fx-mtl
while
arrow-mtl
is still incubating.
We can save ourselves a lot of headache. I think breaking changes in
arrow-fx-mtl
and
arrow-mtl
are now an issue for
0.10.x
. @Jannis that would allow us to merge https://github.com/arrow-kt/arrow/pull/2014 as is. cc\\ @raulraja @pakoito
r
makes sense
p
make mtl depend on fx?
s
No, keep everything as is
r
I think so because a top level lib should not depend on one in the incubator as a rule of thumb either since we are having more relaxed policies with those.
s
but
arrow-fx-mtl
moves out of
arrow-fx
repo and into
ARROW-INCUBATOR repository
along with
arrow-mtl
and
arrow-mtl-data
p
we could rename arrow-fx-mtl to arrow-mtl-fx to understand that the dependency goes the other way
r
@simon.vergauwen, are you meaning about moving
arrow-fx-mtl
to
arrow-incubator
repository?
s
@pakoito yes, that’s also a good point!
But then it makes even more sense to move into
arrow-incubator
👍 1
@Rachel yes
r
arrow-mtl
will be in
arrow-incubator
repository
ok, thanks Simon!
p
arrow-mtl and arrow-mtl-fx into incubator
going once
going twice
SOLD
👍 3
s
haha
Alright let’s get that branch green 🎉
r
Summary:
• Move
arrow-fx-mtl
to
arrow-incubator
repository
• Rename
arrow-fx-mtl
by
arrow-mtl-fx
Right? 👍
👍 2
@pakoito,
arrow-mtl
is already in
arrow-incubator
repository
Thank you all!! I updated the list 😉
BTW, about the rename, what will we do with the artifacts already published with
arrow-fx-mtl
?
p
are they snapshot or 0.10?
if the later, then deprecation notice in 0.10.5 or in the patchnotes for 0.11
also, arrow-mtl-fx should move to arrow-incubator
both, @pakoito
p
gñe
patchnotes it is