Skip to content
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

Scala 3 support #225

Open
vigoo opened this issue Aug 28, 2022 · 5 comments · May be fixed by #246
Open

Scala 3 support #225

vigoo opened this issue Aug 28, 2022 · 5 comments · May be fixed by #246
Assignees

Comments

@vigoo
Copy link
Contributor

vigoo commented Aug 28, 2022

No description provided.

@vigoo
Copy link
Contributor Author

vigoo commented Nov 8, 2022

Depends on zio/zio-schema#395

@vigoo vigoo self-assigned this Dec 7, 2022
@DanielBlanco
Copy link

Any news on this? looks like zio-schema already supports Scala 3.

@vigoo
Copy link
Contributor Author

vigoo commented Feb 15, 2023

It's work in progress: #246

I'm blocked on some weird issues. The most recent one:

I noticed noticed earlier that in zio-flow some zio-prelude Newtypes are not working with Scala 3 - their .apply macro is not type checking, Scala 3 says it has type Any and not the actual type. But it only happened in some tests and in other places in the codebase they worked.
Now I discovered the following:
The newtypes are defined in zioFlow's main sourceset which is a cross-project (jvm/js - probably does not matter)
The non-compiling tests are in an other module, zioFlowRuntime which was using sbt's dependsOn(zioFlowJvm % "compile->compile;test->test") syntax because it shared some generators defined from zioFlow's test source set (this is jvm only)
if I duplicate the shared things in zioFlowRuntime's test source set, not touching the newtype definitions at all, and do dependsOn(zioFlowJVM) then it works (fails with some implicit errors unrelated to the newtypes, to be fixed)

@fabiorizzello
Copy link

Any news?

@vigoo
Copy link
Contributor Author

vigoo commented Mar 31, 2024

I stopped working on this a long time ago. There is a draft PR #246 if anyone wants to pick it up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants