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

Contributing: Importing Scala CLI source into Metals fails #2827

Closed
keynmol opened this issue Mar 29, 2024 · 3 comments
Closed

Contributing: Importing Scala CLI source into Metals fails #2827

keynmol opened this issue Mar 29, 2024 · 3 comments
Labels
bug Something isn't working IDE internal Internal or build-related changes

Comments

@keynmol
Copy link
Contributor

keynmol commented Mar 29, 2024

At 1.2.0 tag I can no longer import scala CLI into metals:

2024.03.28 09:43:31 INFO  time: ran 'mill bloopInstall' in 3.63s
2024.03.28 09:43:31 ERROR Mill command failed: /Users/velvetbaldmime/projects/scala-cli/.metals/millw --mill-version
 0.11.7 --import ivy:com.lihaoyi::mill-contrib-bloop: mill.contrib.bloop.Bloop/install
2024.03.28 09:56:34 INFO  Shutting down server
2024.03.28 09:56:34 INFO  shutting down Metals
2024.03.28 09:56:34 INFO  Exiting server

The full error is easy to reproduce:

> ./millw mill.contrib.bloop.Bloop/install
[build.sc] [47/53] zincReportCachedProblems
java.util.NoSuchElementException: key not found: List(2.13.13)
    scala.collection.MapOps.default(Map.scala:274)
    scala.collection.MapOps.default$(Map.scala:273)
    scala.collection.AbstractMapView.default(MapView.scala:186)
    scala.collection.MapOps.apply(Map.scala:176)
    scala.collection.MapOps.apply$(Map.scala:175)
    scala.collection.AbstractMapView.apply(MapView.scala:186)
    mill.define.Cross.apply(Cross.scala:372)
    millbuild.build$Scala3Graal.$anonfun$resources$7(build.sc:621)

I have bisected it to 98c8e2a (cc @Gedochao).

Not sure if it's a build perk, or mill-contrib-bloop problem, but this prevented me from taking a look at #2708 which I would very much like to solve actually let me try mill-bsp first.

@Gedochao
Copy link
Contributor

cc @tgodzik

@Gedochao
Copy link
Contributor

@keynmol can you try with the latest main branch? Seems like the changes from #2825 should fix this.

@Gedochao Gedochao added bug Something isn't working IDE internal Internal or build-related changes labels Mar 29, 2024
@keynmol
Copy link
Contributor Author

keynmol commented Mar 29, 2024

Thanks folks, it is fixed on lastest main :)

@keynmol keynmol closed this as completed Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working IDE internal Internal or build-related changes
Projects
None yet
Development

No branches or pull requests

2 participants