Skip to content

Commit 38242ee

Browse files
tgodzikWojciechMazur
authored andcommitted
bugfix: Exclude newer LSP4j, which is built on JDK 11
It seems the CI started failing because of out bump, but that was not picked up aside from windows tests weirdly. Metals recently updated to newest LSP4j, though I think nothing new was added and no changes were done to the presentation compiler so we should be ok to downgrade locally: https://github.com/scalameta/metals/pull/6126/files The update was mostly done so that we done fall behind in the LSP standard, but this might become a problem in the future here. Do we ever plan do drop supporting JDK 8? [Cherry-picked 790bee9]
1 parent 941da36 commit 38242ee

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

project/Build.scala

+4-2
Original file line numberDiff line numberDiff line change
@@ -1136,8 +1136,10 @@ object Build {
11361136
libraryDependencies ++= Seq(
11371137
"org.lz4" % "lz4-java" % "1.8.0",
11381138
"io.get-coursier" % "interface" % "1.0.18",
1139-
"org.scalameta" % "mtags-interfaces" % mtagsVersion,
1140-
"com.google.guava" % "guava" % "33.2.1-jre"
1139+
("org.scalameta" % "mtags-interfaces" % mtagsVersion)
1140+
.exclude("org.eclipse.lsp4j","org.eclipse.lsp4j")
1141+
.exclude("org.eclipse.lsp4j","org.eclipse.lsp4j.jsonrpc"),
1142+
"org.eclipse.lsp4j" % "org.eclipse.lsp4j" % "0.20.1",
11411143
),
11421144
libraryDependencies += ("org.scalameta" % "mtags-shared_2.13.12" % mtagsVersion % SourceDeps),
11431145
ivyConfigurations += SourceDeps.hide,

0 commit comments

Comments
 (0)