Add zsh fish bash completion support

[niT-Tin]

Add zsh fish bash completion support

Place the completion file of the corresponding shell in the completion folder in the specified location

[muniu-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: niT-Tin

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[niT-Tin]

and the specific effect is as follows
bash

zsh

fish

[nim65s]

Hi,

What is the point of adding those generated files here ?

I think this should be closed in favor of #861, or something like that.

[niT-Tin]

Oh, thank you for your reminder. I did not notice that there was already a completion operation, but using LSD on my computer did not have this effect, so I wrote these shell completion myself. If there is no problem running lsd completion on someone else, then this pr can indeed be closed

[nim65s]

Arch already provide them : https://gitlab.archlinux.org/archlinux/packaging/packages/lsd/-/blob/main/PKGBUILD#L42-L44
Nix too: https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/misc/lsd/default.nix#L28

You probably just need to set SHELL_COMPLETIONS_DIR

[niT-Tin]

Arch already provide them : https://gitlab.archlinux.org/archlinux/packaging/packages/lsd/-/blob/main/PKGBUILD#L42-L44 Nix too: https://github.com/NixOS/nixpkgs/blob/master/pkgs/tools/misc/lsd/default.nix#L28

You probably just need to set SHELL_COMPLETIONS_DIR

Wow, thanks, I'll try

[zwpaper]

lsd/build.rs

Lines 32 to 35 in 8acaabe

	generate_to(Bash, &mut app, bin_name, &outdir).expect("Failed to generate Bash completions");
	generate_to(Fish, &mut app, bin_name, &outdir).expect("Failed to generate Fish completions");
	generate_to(Zsh, &mut app, bin_name, &outdir).expect("Failed to generate Zsh completions");
	generate_to(PowerShell, &mut app, bin_name, &outdir)

we already support this

/close

[muniu-bot]

@zwpaper: Closed this PR.

In response to this:

lsd/build.rs

Lines 32 to 35 in 8acaabe

	generate_to(Bash, &mut app, bin_name, &outdir).expect("Failed to generate Bash completions");
	generate_to(Fish, &mut app, bin_name, &outdir).expect("Failed to generate Fish completions");
	generate_to(Zsh, &mut app, bin_name, &outdir).expect("Failed to generate Zsh completions");
	generate_to(PowerShell, &mut app, bin_name, &outdir)

we already support this

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.