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

async company completion #320

Merged
merged 1 commit into from
Jul 2, 2017
Merged

Conversation

hypercross
Copy link
Contributor

My project has some insane completion times and it looks like the completions are not async with company. As the features appear to be already present, I tweaked this for myself...

Never worked with elisp before so...

My project has some insane completion times and it looks like the completions are not async with company. As the features appear to be already present, I tweaked this for myself..
@razzmatazz
Copy link
Contributor

Hi @hypercross

Thanks, this PR seems to be working OK for me! Great work.

Would you be able to fix unit tests, or do you need help from me?

@hypercross
Copy link
Contributor Author

I'm not quite certain about project management decency here, so I'd rather leave unit tests to you, thanks!

@razzmatazz
Copy link
Contributor

Ok, fixed those for ya! Thanks!

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