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

Publish for Scala 3 + Native #511

Closed
wants to merge 7 commits into from

Conversation

armanbilge
Copy link
Contributor

Supersedes/backports (?) #477.

This PR is based against 92f3ad9 aka v0.7.29. This is so that these changes can be released as 0.7.30, instead of a milestone of 1.0.0 which the ecosystem has not yet adopted.

Since there are conflicts with subsequent changes on main, I'll need a series/0.7 branch to target this to.

Kamil Podsiadlo and others added 4 commits April 8, 2022 14:01
Drop 3.0.x and pretend it never existed, use 3.1.x for JVM, JS and Native platforms
Scala 3 doesn't allow to call method with parentheses without them. Normally class can extends trait/interface and overwrite their def's using val/override val no matter if def was declared with or without parentheses. However for Scala 3 this is no llonger true. This PR uses a workaround with underscores.
Co-authored-by: Sébastien Doeraene <sjrdoeraene@gmail.com>
@armanbilge armanbilge mentioned this pull request Apr 8, 2022
addSbtPlugin("org.scala-native" % "sbt-scala-native" % "0.4.0")
addSbtPlugin("org.scala-js" % "sbt-scalajs" % "1.6.0")
addSbtPlugin("org.scala-native" % "sbt-scala-native" % "0.4.4")
addSbtPlugin("org.scala-js" % "sbt-scalajs" % "1.10.0")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scala.js 1.10 is forward+backward compatible through Scala.js 1.8. So downstreams can stay on 1.8 if they wish and are not forced to upgrade.

armanbilge and others added 3 commits April 8, 2022 14:35
Co-authored-by: Sébastien Doeraene <sjrdoeraene@gmail.com>
@kpodsiad
Copy link
Member

kpodsiad commented Apr 9, 2022

@armanbilge https://github.com/scalameta/munit/tree/series/0.7 is at your disposal.

@armanbilge armanbilge changed the base branch from main to series/0.7 April 9, 2022 11:36
@armanbilge armanbilge closed this Apr 9, 2022
@armanbilge armanbilge reopened this Apr 9, 2022
@armanbilge
Copy link
Contributor Author

Green! Although credit where credit's due, I didn't contribute a single change to this PR 😂

@armanbilge
Copy link
Contributor Author

On Discord we decided instead of backporting to 0.7.x it's time to cut a stable 1.0. So closing in favor of the other PR.
https://discord.com/channels/632642981228314653/671278431102697472/962322218652692572

@armanbilge armanbilge closed this Apr 9, 2022
jodersky added a commit to jodersky/ScalaPB that referenced this pull request Aug 6, 2022
Since 0.4.4, Scala Native supports Scala 3.

The update to the munit testing library was necessary because the 0.x
series does not support Scala Native on 3 [1].

[1]: scalameta/munit#511
thesamet pushed a commit to scalapb/ScalaPB that referenced this pull request Aug 6, 2022
Since 0.4.4, Scala Native supports Scala 3.

The update to the munit testing library was necessary because the 0.x
series does not support Scala Native on 3 [1].

[1]: scalameta/munit#511
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 this pull request may close these issues.

2 participants