-
Notifications
You must be signed in to change notification settings - Fork 207
Use LSP MarkupContent for generated documentation #1181
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, for my trailing whitespace. Will configure my editor accordingly.
The appveyor fail looks like a transient networking error, rather than an actual fail. And the linux equivalent passes. So I am going to merge (and hold thumbs). |
I'm not sure if this or #1189 is the cause, but after the 0.9.0.0 update, |
I don't have that problem, and I am wondering if its because my emacs config explicitly pulls in ;; ---------------------------------------------------------------------
;; hie lsp stuff
(package-install 'dash-functional)
(package-install 'haskell-mode)
(package-install 'markdown-mode)
(require 'markdown-mode)
(package-install 'ht)
(package-install 'company-lsp)
(package-install 'yasnippet)
(require 'yasnippet)
;; (add-to-list 'load-path "/home/alanz/mysrc/github/alanz/lsp-mode")
(package-install 'lsp-mode)
;; (add-to-list 'load-path "/home/alanz/mysrc/github/alanz/lsp-ui")
(package-install 'lsp-ui)
;; (add-to-list 'load-path "/home/alanz/mysrc/github/alanz/lsp-haskell")
(package-install 'lsp-haskell)
;; New style client ;;;;;;;;;;;;;;
(require 'lsp)
(require 'lsp-haskell)
(add-hook 'haskell-mode-hook #'lsp)
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
;; ---- From
;; https://github.com/tigersoldier/company-lsp -------
;; http://company-mode.github.io/
;(package-install 'company-mode)
;(require 'company-mode)
(add-hook 'after-init-hook 'global-company-mode)
(require 'company-lsp)
(push 'company-lsp company-backends) |
Ahh, so after updating So |
See also emacs-lsp/lsp-ui#172 |
The affects the
HoverReply
message.All documentation is now returned in markdown, and it renders much better in
emacs-lsp
.