Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: vim.pretty_print is deprecated (^neovim-0.9.0) #880

Merged
merged 2 commits into from
May 5, 2023
Merged

fix: vim.pretty_print is deprecated (^neovim-0.9.0) #880

merged 2 commits into from
May 5, 2023

Conversation

ssayin
Copy link
Contributor

@ssayin ssayin commented May 5, 2023

see:
https://github.com/neovim/neovim/blob/040f1459849ab05b04f6bb1e77b3def16b4c2f2b/runtime/doc/news.txt#LL321C3-L321C3

vim.pretty_print is removed in development (prerelease) build: https://github.com/neovim/neovim/releases/tag/nightly

This commit should guarantee backwards compatibility.

Related pull request:
#876

@ssayin ssayin marked this pull request as ready for review May 5, 2023 13:19
@L3MON4D3
Copy link
Owner

L3MON4D3 commented May 5, 2023

TY!
I've taken the liberty of adding a test, and squashing your commits, is this patch fine with you?

@ssayin
Copy link
Contributor Author

ssayin commented May 5, 2023

Hello. Thank you for reviewing my pull request. I have looked at the changes in your patch, and they look fine to me. Please feel free to proceed with squashing the commits and merging the changes.

@L3MON4D3
Copy link
Owner

L3MON4D3 commented May 5, 2023

Nice, thank you👍👍

@L3MON4D3 L3MON4D3 merged commit b4bc24c into L3MON4D3:master May 5, 2023
@ssayin ssayin deleted the patch-1 branch May 5, 2023 22:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants