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
Cross-build for Scala 2.13 #162
Conversation
def scala3Version = "3.1.2" | ||
def scala3Version = "3.2.0-RC2" |
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.
Had to bump to v3.2.0 since I was hitting some compiler bugs in 3.1.x.
Seems like this PR is on good way @armanbilge ;) |
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.
scalafmt and native fails on CI, after they're green I think this will be good to go 👍
@@ -36,13 +37,11 @@ inThisBuild( | |||
|
|||
ThisBuild / scalafixDependencies += "com.github.liancheng" %% "organize-imports" % "0.6.0" | |||
|
|||
lazy val core = crossProject(JSPlatform, JVMPlatform, NativePlatform) | |||
lazy val core = crossProject(JSPlatform, JVMPlatform /*, NativePlatform*/ ) |
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.
Is there a reason for commenting native version?
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.
Yes, because I had to bump to Scala 3.2.0-RC2 the Native plugin is not yet published for these versions.
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.
CI for for native can be disabled (for a while) to not block this PR
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.
@kpodsiad cool, thanks, I've fixed formatting and disabled Native CI for now.
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.
- for a while - as long as there is no ScalaNative plugin for 3.2.x ;)
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, CI is green 👍
* Cross-compile for 2.13 * Cross-compile integrations tests * Bump to 3.2.0-RC2 * Disable native plugin and temporarily remove native from CI (there is no one for 3.2.x)
Closes #159.
I had a brief look at adding 2.12 as well, but that still requires more changes. 2.13 is pretty good for now :)