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

fix for #1172 #1183

Closed
wants to merge 1 commit into from
Closed

fix for #1172 #1183

wants to merge 1 commit into from

Conversation

andviro
Copy link
Contributor

@andviro andviro commented Jan 24, 2017

Hi!
Newer versions of Neovim expect the on_stdout/on_stderr handler to have the same function signature as the on_exit handler. This caused empty std_combined list and the "invalid argument" error. I've added an extra guard condition on it, just in case.

@fatih
Copy link
Owner

fatih commented Feb 2, 2017

This is now merged with #1176. Thanks @andviro anyways 👍

@fatih fatih closed this Feb 2, 2017
@andviro
Copy link
Contributor Author

andviro commented Feb 2, 2017

All thanks to you, for the excellent plugin. Keep up the great work!

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