[java][lsp] Minor cleanup in lsp-java

- Fixed setting dap-java shortcuts - ATM the shortcuts were not enabled due to
the fact that dap-mode configuration was overriding them since it was running
after this piece of code
- removed configuration code about lsp-java since it was either about
missing properties or about setting defaults already set by lsp-mode/lsp-java
(e. g. lsp-eldoc-render-all is now nil by default)
This commit is contained in:
Ivan Yonchovski 2019-12-29 11:42:00 +02:00 committed by duianto
parent a337d55102
commit 3771759ddb
3 changed files with 13 additions and 17 deletions

View file

@ -2138,6 +2138,9 @@ Other:
- Fixed prefixes for =java-mode= and =gradle-mode= (thanks to Seong Yong-ju)
- Enabled =lsp-java= when the =java-backend= is =nil= and the =lsp= layer is
enabled (thanks to Ivan Yonchovski)
- Fixed =dap-java= shortcuts (thanks to Ivan Yonchovski)
- Removed =lsp-java= configurations for missing properties or for defaults
that already are set by =lsp-mode/lsp-java= (thanks to Ivan Yonchovski)
**** Javascript
- Improvements:
- Leverage js-doc Yasnippet integration if available (thanks to Andriy Kmit')

View file

@ -133,7 +133,15 @@
(defun spacemacs//java-setup-lsp-dap ()
"Setup DAP integration."
(require 'dap-java))
(require 'dap-java)
(spacemacs/set-leader-keys-for-major-mode 'java-mode
;; debug
"ddj" 'dap-java-debug
"dtt" 'dap-java-debug-test-method
"dtc" 'dap-java-debug-test-class
;; run
"tt" 'dap-java-run-test-method
"tc" 'dap-java-run-test-class))
(defun spacemacs//java-setup-lsp-flycheck ()
"Setup LSP Java syntax checking."

View file

@ -213,22 +213,7 @@
"cc" 'lsp-java-build-project
"cp" 'lsp-java-spring-initializr
"an" 'lsp-java-actionable-notifications
;; dap-mode
;; debug
"ddj" 'dap-java-debug
"dtt" 'dap-java-debug-test-method
"dtc" 'dap-java-debug-test-class
;; run
"tt" 'dap-java-run-test-method
"tc" 'dap-java-run-test-class)
(setq lsp-highlight-symbol-at-point nil
lsp-ui-sideline-update-mode 'point
lsp-eldoc-render-all nil
lsp-java-completion-guess-arguments t))))
"an" 'lsp-java-actionable-notifications))))
(defun java/init-mvn ()
(use-package mvn