Skip to content

Dependency updates 2020-07#1218

Merged
alexandru merged 5 commits into
monix:masterfrom
alexandru:dependency-updates-202007
Jul 26, 2020
Merged

Dependency updates 2020-07#1218
alexandru merged 5 commits into
monix:masterfrom
alexandru:dependency-updates-202007

Conversation

@alexandru
Copy link
Copy Markdown
Member

@alexandru alexandru commented Jul 22, 2020

The problematic change I'm introducing is that JCTools is now a "shaded" library, meaning that it's getting published under a new monix-internal-jctools artifact and its classes moved to monix.execution.internal.jctools.

Making this change because other libraries depend on JCTools and we can end up in conflict with them.

Will have to test the generated artifacts.

@alexandru alexandru changed the title Dependency updates 2020-07 WIP: Dependency updates 2020-07 Jul 22, 2020
@Avasil
Copy link
Copy Markdown
Collaborator

Avasil commented Jul 22, 2020

BTW I have just updated to Scala 2.13.3 in #1217
There were lots of errors about lack of ()

Comment thread project/plugins.sbt
addSbtPlugin("org.scalameta" % "sbt-scalafmt" % "2.4.0")
addSbtPlugin("com.github.tkawachi" % "sbt-doctest" % "0.9.6")
addSbtPlugin("org.scoverage" % "sbt-scoverage" % "1.6.1")
val scalaJSVersion = Option(System.getenv("SCALAJS_VERSION")).filter(_.nonEmpty).getOrElse("1.1.1")
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm not sure we want to do this: https://www.scala-js.org/news/2020/05/18/announcing-scalajs-1.1.0/

It is backward binary compatible with all earlier versions in the 1.x series: libraries compiled with 1.0.x can be used with 1.1.0 without change.
It is not forward binary compatible with 1.0.x: libraries compiled with 1.1.0 cannot be used with 1.0.x.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yikes, that sucks.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

So I think we should go ahead with 1.1.1. It's not the same restrictions as with the JVM, because the Scala.js compiler is needed when swapping a JAR anyway and from what I understand it will complain if the library isn't compatible.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@alexandru
Copy link
Copy Markdown
Member Author

@Avasil I'll wait for that PR then, as I also wanted to update Scala and preferably to add the sbt-tpolecat plugin.

@alexandru alexandru changed the title WIP: Dependency updates 2020-07 Dependency updates 2020-07 Jul 26, 2020
@alexandru alexandru changed the title Dependency updates 2020-07 WIP: Dependency updates 2020-07 Jul 26, 2020
@alexandru alexandru changed the title WIP: Dependency updates 2020-07 Dependency updates 2020-07 Jul 26, 2020
@alexandru alexandru changed the title Dependency updates 2020-07 WIP: Dependency updates 2020-07 Jul 26, 2020
@alexandru alexandru changed the title WIP: Dependency updates 2020-07 Dependency updates 2020-07 Jul 26, 2020
@alexandru
Copy link
Copy Markdown
Member Author

I'm also preparing a PR activating sbt-tpolecat and fixing a ton of compiler warnings 😐

I will merge this one for now. If you disagree with the shading, let me know.

The artifacts need to be tested anyway, so we'll have to publish a sha release.

@alexandru alexandru merged commit 6500f95 into monix:master Jul 26, 2020
@alexandru alexandru deleted the dependency-updates-202007 branch July 26, 2020 09:13
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