Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RDY] lsp: make showMessage and logMessage callbacks different #11942

Merged
merged 1 commit into from
Mar 2, 2020

Conversation

h-michael
Copy link
Contributor

According to the LSP specification, showMessage is what is displayed and logMessage is what is stored.
Since these are different things, I divide the callback into those that match.

@h-michael h-michael changed the title lsp: make showMessage and logMessage callbacks different [RDY] lsp: make showMessage and logMessage callbacks different Mar 1, 2020
@marvim marvim added the RDY label Mar 1, 2020
@teto
Copy link
Member

teto commented Mar 2, 2020

rebase please

According to the LSP specification, showMessage is what is displayed and logMessage is what is stored.
Since these are different things, I devide the callback into those that match.
@h-michael
Copy link
Contributor Author

@teto I've rebased :)

@teto teto merged commit fbc4c4f into neovim:master Mar 2, 2020
@h-michael h-michael deleted the lsp-log-message branch March 2, 2020 13:28
ataul443 pushed a commit to ataul443/neovim that referenced this pull request Mar 5, 2020
According to the LSP specification, showMessage is what is displayed and logMessage is what is stored.
Since these are different things, I devide the callback into those that match.
@dundargoc dundargoc removed the request for review from justinmk October 3, 2022 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants