Merge pull request #1847 from filalex77/lang-scala-ensime-metals

lang/scala: replace ensime with metals

modules/lang/scala/config.el

@@ -15,23 +15,6 @@
     (add-hook 'scala-mode-local-vars-hook #'lsp!)))
 
 
-(use-package! ensime
-  :unless (featurep! +lsp)
-  :defer t
-  :config
-  (setq ensime-startup-snapshot-notification nil
-        ensime-startup-notification nil
-        ensime-eldoc-hints 'all
-        ;; let DOOM handle company setup
-        ensime-completion-style nil)
-
-  (set-company-backend! 'scala-mode '(ensime-company company-yasnippet))
-
-  ;; Fix void-variable imenu-auto-rescan error caused by `ensime--setup-imenu'
-  ;; trying to make imenu variables buffer local before imenu is loaded.
-  (require 'imenu))
-
-
 (use-package! sbt-mode
   :after scala-mode
   :config (set-repl-handler! 'scala-mode #'+scala/open-repl))

modules/lang/scala/doctor.el

@@ -0,0 +1,9 @@
+;;; lang/scala/doctor.el -*- lexical-binding: t; -*-
+
+(assert! (or (not (featurep! +lsp))
+             (featurep! :tools lsp))
+         "This module requires (:tools lsp)")
+
+(if (and (featurep! +lsp)
+         (not (executable-find "metals-emacs")))
+    (warn! "metals-emacs isn't installed"))

modules/lang/scala/packages.el

@@ -3,9 +3,3 @@
 
 (package! sbt-mode)
 (package! scala-mode)
-
-(unless (featurep! +lsp)
-  ;; Fix #1697: Ensime doesn't have a master branch and its MELPA recipe doesn't
-  ;; specify a branch. Straight can't handle packages with non-standard primary
-  ;; branches (at the time of writing), so we must specify it manually:
-  (package! ensime :recipe (:branch "2.0")))