-
Notifications
You must be signed in to change notification settings - Fork 62
Scala 3 migration #270
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 migration #270
Conversation
Can we try to merge it together . I feel that we need some deeper domain knowledge here 🙏 |
@stephenamar-db any chances we can make this cross compile for Scala 3 ? |
Thanks, I just found I need this, because one of my dependencies is using Scala 3.3.5:) |
I will gladly review this |
I've branched the pre-scala3 version into the branch 0.4-series. master will be for the work to get it to run with scala3. Once it's stable, we'll say good bye to 0.4.x. |
fa13998
to
32c9904
Compare
env: | ||
# Set LANG=C to simulate least-common-denominator target deployment environments: | ||
LANG: C | ||
SCALANATIVE_THREAD_STACK_SIZE: 10m |
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.
note: to prevent stackoverflow in native tests scala-native/scala-native#4253
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.
Aside from the .toSeq bit, I don't see any major changes.
Can't wait to have a fresh new release with this 😍 |
Co-authored-by: He-Pin(kerr) <[email protected]>
Co-authored-by: He-Pin(kerr) <[email protected]>
Co-authored-by: He-Pin(kerr) <[email protected]>
Co-authored-by: He-Pin(kerr) <[email protected]>
I tried to migrate code to scala 3. This required updates of some libs and a little code adjustments.
I would appreciate your input 🙏