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

Disable BSP in Jmh and vscode-dotty #11687

Merged
merged 2 commits into from
Mar 10, 2021
Merged

Disable BSP in Jmh and vscode-dotty #11687

merged 2 commits into from
Mar 10, 2021

Conversation

adpi2
Copy link
Contributor

@adpi2 adpi2 commented Mar 10, 2021

  1. Disable BSP in Jmh config

sbt treats the Jmh configuration as a build target because it contains the sbt.Defaults.configSettings (sources, compile, scalaInstance...). The buildTarget/sources request triggers the project compilation to generate the benchmarking files. So the project import can fail if the compilation fails. To avoid this unpleasant situation we can disable BSP in Jmh.

  1. Disable BSP in vscode-dotty project

This is not a Scala project so it doesn't make sense to import it via BSP.

@adpi2 adpi2 requested a review from smarter March 10, 2021 15:54
@smarter smarter enabled auto-merge March 10, 2021 16:03
@smarter smarter merged commit 2961f69 into scala:master Mar 10, 2021
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.

2 participants