Skip to content

Add completion for azure cli (az)#8141

Merged
faho merged 2 commits into
fish-shell:masterfrom
kfkonrad:feature/az-completion
Jul 21, 2021
Merged

Add completion for azure cli (az)#8141
faho merged 2 commits into
fish-shell:masterfrom
kfkonrad:feature/az-completion

Conversation

@kfkonrad

Copy link
Copy Markdown
Contributor

Description

This introduces completions for the Azure CLI az. az uses the argcomplete library for its completions and the code I've used is little more than a boilerplate snippet to access those completions.

argcomplete has been compatible with fish for over 2 years now (argcomplete!260), but this is (to my knowledge) the first time this is being added to fish-shell directly. The code is basically copied from a snippet mentioned in argcomplete!260, except I replaced myscript with az.

TODOs:

  • Changes to fish usage are reflected in user documentation/manpages.
  • Tests have been added for regressions fixed
  • User-visible changes noted in CHANGELOG.rst

Comment thread share/completions/az.fish Outdated
Comment thread share/completions/az.fish Outdated
Comment thread share/completions/az.fish Outdated
Comment thread share/completions/az.fish Outdated
Comment thread CHANGELOG.rst Outdated
@kfkonrad kfkonrad force-pushed the feature/az-completion branch from 63e78fc to 1d012b6 Compare July 21, 2021 05:58
@faho faho merged commit 869c42d into fish-shell:master Jul 21, 2021
@faho

faho commented Jul 21, 2021

Copy link
Copy Markdown
Member

Merged, thanks!

@kfkonrad kfkonrad mentioned this pull request Jul 28, 2021
3 tasks
thunder-coding pushed a commit to thunder-coding/fish-shell that referenced this pull request Jul 28, 2021
* add completion for azure cli (az)

* alter az.fish to match faho's requests

Co-authored-by: Kevin Konrad <kevin.konrad@unicepta.com>
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants