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

Add async and async_wrap from telescope pr #46

Closed
wants to merge 1 commit into from

Conversation

oberblastmeister
Copy link
Collaborator

from pr

@kkharji
Copy link
Contributor

kkharji commented Jan 5, 2021

😍 a place where we can have useful abstraction over loop. NICE!!

@kkharji
Copy link
Contributor

kkharji commented Jan 5, 2021

simple @Usage examples will be appreciate it as well as tests look at /tests/auto/curl_spec.lua for inspiration

@tjdevries
Copy link
Member

I need to read up a bit more on the actual implementation of this and how Neovim is embedding libuv again before I can merge (since we may want to do some other things here).

We should also add safe_async to check the type of the arugments passes so that we cannot crash the editor :) the other can be use at your own risk.

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