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

Replace deprecated pseudo-private API usage #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alerque
Copy link

@alerque alerque commented Feb 18, 2021

The pseudo-private _VimuxOption function has been deprecated and an
official public VimuxOption method is available. However the way
defaults are handled is also changed. If you don't want to use the
default value, just don't. Hard coding the prompt here seems to make
more sense that even bothering to ask what the option value is.

The pseudo-private _VimuxOption function has been deprecated and an
official public VimuxOption method is available. However the way
defaults are handled is also changed. If you don't want to use the
default value, just don't. Hard coding the prompt here seems to make
more sense that even bothering to ask what the option value is.
@michalwski
Copy link

@spiegela any chance of getting this PR merged? Looks like it can help with currently not working VimixPromptCommand.

The error I'm getting with newest vimix plugin is:

Error detected while processing function VimixPromptCommand:
line    1:
E117: Unknown function: _VimuxOption
E116: Invalid arguments for function input
E15: Invalid expression: input(_VimuxOption("g:VimixPromptString", "Mix Command? "))
line    2:
E121: Undefined variable: l:command
E116: Invalid arguments for function <SNR>57_VimixRunCommand

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