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

flag suggestions like --verbose get triggered by two dashes --, but insert four ---- #846

Closed
j-lakeman opened this issue May 14, 2023 · 3 comments · Fixed by #847
Closed
Labels
bug Something isn't working

Comments

@j-lakeman
Copy link

Code editor

Visual Studio Code

Platform

Linux

Version

v1.36.0

What steps will reproduce the bug?

Insert a command that has the --verbose flag, like rsync, for example. Continue typing --v, until the autocompletion offers --version and --verbose. Pick either one from the menu. Two additional dashes get inserted, so that there are four altogether.

How often does it reproduce? Is there a required condition?

reproduces for any -- flags

What is the expected behavior?

don't insert additional dashes

What do you see instead?

Two additional dashes get inserted, so that there are four altogether.
This is unintended behaviour.

Additional information

video.webm
@skovhus
Copy link
Collaborator

skovhus commented May 15, 2023

Thanks for reporting this!

A fix for this was released in the vscode extension version 1.37.0.

@skovhus skovhus added the bug Something isn't working label May 15, 2023
@j-lakeman
Copy link
Author

Cheers, seems resolved! But I'm experiencing another issue now. The first letter of the accepted flag gets removed, see video below:

video.webm

@lxr2010
Copy link
Contributor

lxr2010 commented May 30, 2023

Cheers, seems resolved! But I'm experiencing another issue now. The first letter of the accepted flag gets removed, see video below:

video.webm

I'm using nvim.cmp and bash-language-server, also have this problem.
It seems that nvim.cmp deals with "insertText" field differently from VS code client.
I tried to describe insertion with "textEdit" instead of "insertText" , recompiled and reinstalled bash-language-server, and the problem solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants