Skip to content

Conversation

@phanen
Copy link
Contributor

@phanen phanen commented Feb 23, 2025

No description provided.

@maskudo
Copy link
Owner

maskudo commented Feb 23, 2025

Seems to have a some merge conflict. Please resolve them and let me know when you are done with it.

@maskudo
Copy link
Owner

maskudo commented Feb 23, 2025

Many vimscript functions cannot run inside lua loops. Are you sure vim.fn.mkdir doesnt suffer from this?

@phanen
Copy link
Contributor Author

phanen commented Feb 23, 2025

Sorry it's not marked as fast, a quick test:

vim.uv.new_timer():start(0, 0, fn.mkdir)

I find someone have been make a compat here. So I remove it, although it would be better to have a vim.fs.mkdir with p option here that wrap libuv...

@maskudo
Copy link
Owner

maskudo commented Feb 23, 2025

The PR looks good. Is there anything else you are working on in this PR or should i merge it? @phanen

@phanen
Copy link
Contributor Author

phanen commented Feb 23, 2025

Nothing more, thanks for review (:

@maskudo maskudo merged commit eee96fe into maskudo:main Feb 23, 2025
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