Use .documentationPopups as the sole determiner for LSP hover

InSyncWithFoo · Oct 8, 2024 · c4f385c · c4f385c
1 parent 5fbf06c
commit c4f385c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/main/kotlin/insyncwithfoo/ryecharm/ruff/lsp/RuffServerDescriptor.kt b/src/main/kotlin/insyncwithfoo/ryecharm/ruff/lsp/RuffServerDescriptor.kt
@@ -22,7 +22,7 @@ internal class RuffServerDescriptor(project: Project, private val executable: Pa
     override val lspCompletionSupport = null
     override val lspCommandsSupport = null
 
-    override val lspHoverSupport = configurations.run { documentationPopups && documentationPopupsForNoqaComments }
+    override val lspHoverSupport = configurations.documentationPopups
     override val lspDiagnosticsSupport = DiagnosticsSupport(project).takeIf { configurations.linting }
     override val lspCodeActionsSupport = CodeActionsSupport(project).takeIf { configurations.quickFixes }
     override val lspFormattingSupport = FormattingSupport(project).takeIf { configurations.formatting }

diff --git a/src/main/kotlin/insyncwithfoo/ryecharm/ruff/lsp4ij/HoverFeature.kt b/src/main/kotlin/insyncwithfoo/ryecharm/ruff/lsp4ij/HoverFeature.kt
@@ -14,6 +14,6 @@ internal class HoverFeature : LSPHoverFeature() {
     private val configurations = project.ruffConfigurations
 
     override fun isEnabled(file: PsiFile) =
-        configurations.run { documentationPopups && documentationPopupsForNoqaComments }
+        configurations.documentationPopups
 
 }