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

feat(intergrations.otter): update to use otter 2.0 #1505

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

benlubas
Copy link
Contributor

@benlubas benlubas commented Jul 7, 2024

Otter.nvim got a pretty large update.

TL;DR: it's a language server now, so no need to custom keybinds (normal LSP keybinds are used)

@benlubas benlubas marked this pull request as draft July 7, 2024 21:11
@benlubas
Copy link
Contributor Author

benlubas commented Jul 7, 2024

Back to draft as there is a bug causing inconsistent completions.

@vhyrro
Copy link
Member

vhyrro commented Jul 8, 2024

Happy to see otter being improved. If you could explain the bug I can also give it a look.

@benlubas
Copy link
Contributor Author

benlubas commented Jul 8, 2024

jmbuhr/otter.nvim#159

Perhaps you will have better luck finding a consistent reproduction. Though as jmbuhr mentions in that thread, it might be an issue with nvim or pyright.

@benlubas benlubas marked this pull request as ready for review July 14, 2024 14:48
@benlubas
Copy link
Contributor Author

@vhyrro I'm okay with merging this. I can't reproduce it anymore

@vhyrro
Copy link
Member

vhyrro commented Jul 15, 2024

Hah sounds good to me!

@vhyrro vhyrro merged commit 1347eeb into nvim-neorg:main Jul 15, 2024
5 of 6 checks passed
@benlubas benlubas deleted the push-rmokpvyqqtmr branch July 15, 2024 21:25
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.

2 participants