I'm working on migrating my code base from 1.1 to 1.2 using the script at <https://gist.github.com/n...

# arrow

[Wesley Hartford]

I'm working on migrating my code base from 1.1 to 1.2 using the script at https://gist.github.com/nomisRev/e01ddc354c84b8b626c23d024706b916 and I've found a bug in the script. I don't think I can submit a PR or create an issue for a gist, so here are the details:

[Wesley Hartford]

The script replaces EffectScope< with Raise< before it replaces EagerEffectScope< with Raise<, as a result, any use of EagerEffectScope ends up as EagerRaise rather than just Raise.

[Wesley Hartford]

Easy enough to fix manually, or in the script as needed, but I thought I should let you know.

[Wesley Hartford]

The script also does not update the import arrow.core.continuations.Effect to arrow.core.raise.Effect

[Wesley Hartford]

In addition what I've noted above, I had to add a import arrow.core.raise.toEither in a few places, and remove suspend from a few functions and the project built. I'm working on tidying up some use of deprecation at this point.

[Wesley Hartford]

All things considered, I'm really happy with how things went and the newer DSL style is really nice.

[simon.vergauwen]

Hey @Wesley Hartford,

Thanks for letting me know! I will update the gist. Sadly there are still some issue with OpenRewrite, but they're currently working hard of improving there Kotlin support. I hope to have it finished by the end-of-year before 2.0 lands.

[Wesley Hartford]

Here's the migrate script I ended up with, I think there were a couple other minor changes. All told it went pretty smoothly.

migrate.main.kts