Skip to content

Check for loop or uv for lazypath#1095

Merged
feoh merged 1 commit into
nvim-lua:masterfrom
LehMichael:patch-1
Aug 22, 2024
Merged

Check for loop or uv for lazypath#1095
feoh merged 1 commit into
nvim-lua:masterfrom
LehMichael:patch-1

Conversation

@LehMichael

@LehMichael LehMichael commented Aug 20, 2024

Copy link
Copy Markdown
Contributor

Fixes #1073
This is the suggested way from https://lazy.folke.io/installation and should also work on older nvim versions.

@dam9000

dam9000 commented Aug 20, 2024

Copy link
Copy Markdown
Contributor

@mike-jl this LGTM.
I'd suggest you remove the NOTE block in the PR description, as it makes it look like it's a PR opened by mistake.

@LehMichael

Copy link
Copy Markdown
Contributor Author

Done! Thanks for the heads up!

@feoh feoh merged commit ce0c734 into nvim-lua:master Aug 22, 2024
@VlaDexa

VlaDexa commented Aug 22, 2024

Copy link
Copy Markdown
Contributor

Hi, sorry for noticing this so late, but I think this is just clutter, because our other config stuff like type hints or even plugins like conform don't support older nvim versions and require 0.10, where vim.loop is deprecated

@feoh

feoh commented Aug 23, 2024

Copy link
Copy Markdown
Collaborator

Should I publish a revert?

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.

vim.uv field is nil causing error in init.lua

4 participants