-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix the dependencies of the bootstrapped stdlib #24379
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
Conversation
| // because we allow cross-compilation. | ||
| versionScheme := Some("always"), | ||
| crossPaths := false, | ||
| crossVersion := CrossVersion.disabled, |
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.
@sjrd seems we have broken the artifact name on October 8th when we added the DottyJSPlugin to this project. This here fixes it but is it the correct way?
See the nightly releases: https://repo.scala-lang.org/ui/repos/tree/General/local-maven-nightlies/org/scala-js/scalajs-scalalib_2.13_sjs1_3/3.8.0-RC1-bin-20251008-6194563-NIGHTLY.
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.
Seems to be right way, yes.
…-language-server` sbt doesn't evict a library dependency if it has the same dependency as a project
…js1_3` This broke when we added the a dependency to the `DottyJSPlugin`
7d82fbf to
e1953d8
Compare
Before we used to rely on the presence of the
scala-library:2.13.xin the classpath to copy the binaries. This was not the case for a long time now and we can correctly not depend on it in the bootstrapped library.