-
Notifications
You must be signed in to change notification settings - Fork 126
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
Migrate to bloop-cli
and remove module from scala-cli
#1971
Comments
I'd like to advance this as much as I could this week. Not sure what others think about it, but here is a possible path forward, that I'm working on right now (the first steps at least):
That's only a proposal of a path forward for now, as I guess we'll need to agree on the last steps in particular 😅 But to summarize, I basically propose to make the current Scala CLI Bloop fork the new default Bloop server, to make the Scala CLI Bloop libraries and CLI the new default Bloop libraries and CLI, and to leave only integrations in the current Bloop repo. |
Then in a second time, we could have Metals use the new Bloop library rather than the current launcher module from scalacenter/bloop (basically what scalameta/metals#4103 attempts to do), and remove the launcher and bloopgun modules, and the former CLI, from scalacenter/bloop. |
An update on this:
|
Is your feature request related to a problem? Please describe.
There are currently two different forks. @alexarchambault outlined some of the steps that would be necessary to make these converge and one of them was that
bloop-cli
was moved to its own repo, but it may have fallen out of sync and it's not currently being used by this repo. Instead, this repo just uses it as a module inlined.Describe the solution you'd like
bloop-cli
and the code that is inlined to be synced up.bloop-cli
to be documented and releasedscala-cli
to just rely onbloop-cli
Describe alternatives you've considered
Have multiple forks of bloop out there forever.
Additional context
Once we get this done, we can better explore using it in Metals, which would allow us to drop it in Bloop, simplify it, and then maybe even star the convergence project.
The text was updated successfully, but these errors were encountered: