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

Upgrade to sbt 0.13.14-RC2, switch to build.sbt #2101

Merged
merged 4 commits into from Mar 24, 2017

Conversation

smarter
Copy link
Member

@smarter smarter commented Mar 14, 2017

Let's wait until sbt/sbt#3012 is fixed first.

@felixmulder
Copy link
Contributor

👍 for removing the unused scalastyle plugin

@smarter smarter changed the title [DO NOT MERGE] Upgrade to sbt 0.13.14-RC1 Upgrade to sbt 0.13.14-RC2 Mar 24, 2017
@smarter smarter changed the title Upgrade to sbt 0.13.14-RC2 Upgrade to sbt 0.13.14-RC2, switch to build.sbt Mar 24, 2017
Also replace com.typesafe.sbt:sbt-interface by the equivalent
org.scala-sbt:interface since the former is not published in 0.13.14 for
some reason (sbt/sbt#3012).
Also fix a bug where the compiler output for the tests ended up in the
wrong directory, causing some new tests from 0.13.14 to fail.
Using the same technique than scala-js where we just forward to
Build.scala: scala-js/scala-js#2312
@smarter
Copy link
Member Author

smarter commented Mar 24, 2017

@felixmulder I think this is ready to go in (even though sbt 0.13.14 is not released yet, this way we may be able to get potential issues fixed before the release)

@felixmulder
Copy link
Contributor

What about sbt/sbt#3012?

@felixmulder
Copy link
Contributor

Otherwise LGTM

@smarter
Copy link
Member Author

smarter commented Mar 24, 2017

Using org.scala-sbt:interface is fine even with coursier as long as we add resolvers to all subprojects as explained in coursier/coursier#284 (comment)

@smarter smarter merged commit 90eb9d2 into scala:master Mar 24, 2017
@allanrenucci allanrenucci deleted the upgrade/sbt0.13.14 branch December 14, 2017 19:25
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.

None yet

2 participants