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

Powershell completions coreutils __ completes files. #5933

Closed
LucasLehmann opened this issue Feb 2, 2024 · 4 comments · Fixed by #5949
Closed

Powershell completions coreutils __ completes files. #5933

LucasLehmann opened this issue Feb 2, 2024 · 4 comments · Fixed by #5949

Comments

@LucasLehmann
Copy link

This seems to be caused by powershells completion not being capable of having a paramater value of '' this happens with the completion for seq and causes all completions coreutils __ to not be applied anything else such as coreutils seq __ works and will complete. This stops happening if the string is changed to anything else such as ' ' or 'Print a sequence of numbers'.

@tertsdiepraam
Copy link
Member

tertsdiepraam commented Feb 3, 2024

I honestly do not quite understand what you are trying to say. Is the problem that the argument of coreutils is not being limited to the list of utils? It would also be very helpful if you could figure out whether this is a problem on our side or on the side of clap-complete. Could you maybe show an example of the current behaviour and the desired behaviour?

@LucasLehmann
Copy link
Author

when using powershell tab completion writing coreutils then attempting to tab complete it will fail to complete for utils and only complete for files.
image
this does not apply to any command after just coreutils such as coreutils ls wich always works fine

this seems to be caused by this:
image

if i change the line to:
image

it completes just fine
image

@tertsdiepraam
Copy link
Member

Ah I see now. Thanks for the explanation. Is there a way to fix this without passing a a description? I think this should probably be fixed in clap_complete in the clap repository. Could you open an issue there?

@LucasLehmann
Copy link
Author

Doesn't seem like it is possible, though having it be a space character works. Ill open an issue on the clap repository.

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 a pull request may close this issue.

2 participants