-
Notifications
You must be signed in to change notification settings - Fork 17
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
Support Scala Native + Scala 3 #185
Conversation
Fix most warnings Update Scala 3 to 3.3.1 Update paiges-core to 0.4.3 Update Scala.js to 1.13.2 Update sbt-mdoc to 2.3.7
Move munit-scalacheck to test dependencies Add scalacheck as main dependency since it was used by the Compile scope and got as transitive dependency of munit-scalacheck
Fix scalacOptions for Scala 3 Fix more warnings Remove the various generated lazy val modJVM = mod.jvm
Scala.js doesn't implement java.io.File
@tgodzik The CI is finally green :) Can I ask you a 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.
Looks good! Thanks for looking into this, I haven't had much time to look at metaconfig over the past year.
Should we do a release or do you want to test it out still?
Looks like the release is failing though https://github.com/scalameta/metaconfig/actions/runs/6364431764/job/17280704001 <- anything looks familiar? |
I'll give it a look tomorrow 👍 |
Opened #187 which should fix the problem on publishing |
Fixes #179
Fix most warnings
Update Scala 3 to 3.3.1
Update paiges-core to 0.4.3
Update Scala.js to 1.13.2
Update sbt-mdoc to 2.3.7
Update scala-collection-compat to 2.11.0
Update munit to 1.0.0-M10
Remove munit-scalacheck from main dependencies
Add only scalacheck as a dependency
Update Scala Native to 0.4.15
Add Scala 3 to all projects
Move
java.io.File
specific test tojvm-native
folder