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

Bump coursier version? #66

Open
netvl opened this issue Jan 17, 2018 · 2 comments
Open

Bump coursier version? #66

netvl opened this issue Jan 17, 2018 · 2 comments

Comments

@netvl
Copy link

netvl commented Jan 17, 2018

Currently sbt-scalafmt-coursier depends on Coursier 1.0.0-RC13, while the latest version is Courser 1.0.0. Because of this, if my build depends both on Coursier 1.0.0 and the latest sbt-scalafmt-coursier, I get the following warnings each time SBT is started:

[warn] Found version conflict(s) in library dependencies; some are suspected to be binary incompatible:
[warn]  * io.get-coursier:coursier-cache_2.12:1.0.0 is selected over 1.0.0-RC13
[warn]      +- io.get-coursier:sbt-shared:1.0.0 (scalaVersion=2.12, sbtVersion=1.0) (depends on 1.0.0)
[warn]      +- io.get-coursier:sbt-coursier:1.0.0 (scalaVersion=2.12, sbtVersion=1.0) (depends on 1.0.0)
[warn]      +- com.lucidchart:sbt-scalafmt-coursier:1.15 (scalaVersion=2.12, sbtVersion=1.0) (depends on 1.0.0-RC13)
[warn]  * io.get-coursier:coursier_2.12:1.0.0 is selected over 1.0.0-RC13
[warn]      +- io.get-coursier:coursier-extra_2.12:1.0.0          (depends on 1.0.0)
[warn]      +- io.get-coursier:sbt-shared:1.0.0 (scalaVersion=2.12, sbtVersion=1.0) (depends on 1.0.0)
[warn]      +- io.get-coursier:sbt-coursier:1.0.0 (scalaVersion=2.12, sbtVersion=1.0) (depends on 1.0.0)
[warn]      +- io.get-coursier:coursier-cache_2.12:1.0.0          (depends on 1.0.0)
[warn]      +- com.lucidchart:sbt-scalafmt-coursier:1.15 (scalaVersion=2.12, sbtVersion=1.0) (depends on 1.0.0-RC13)

This problem is not critical, of course, but a nuisance nevertheless. Is there a particular reason to stick to the prerelease version of Coursier?

@ches
Copy link
Contributor

ches commented Feb 1, 2018

See #65. I think we just need a maintainer's attention.

@ches
Copy link
Contributor

ches commented Feb 27, 2018

#65 is merged, just need a release now for these warnings to go away.

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

No branches or pull requests

2 participants