Skip to content

Fix aws-cli completion for ZSH#1627

Open
daeghanelkin wants to merge 1 commit intodevcontainers:mainfrom
daeghanelkin:fix/aws-cli-zsh-completion
Open

Fix aws-cli completion for ZSH#1627
daeghanelkin wants to merge 1 commit intodevcontainers:mainfrom
daeghanelkin:fix/aws-cli-zsh-completion

Conversation

@daeghanelkin
Copy link
Copy Markdown

The ZSH completer script provided by the aws/aws-cli repo is not set up to be used as a completer function, and thus cannot be dropped into a directory like ~/.oh-my-zsh/completions and be expected to work.

This change also moves the completer to a more standard location for system-wide ZSH completions, and adds the necessary header to make it work as a completer.

@daeghanelkin
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@daeghanelkin daeghanelkin marked this pull request as ready for review April 21, 2026 01:54
@daeghanelkin daeghanelkin requested a review from a team as a code owner April 21, 2026 01:54
Copy link
Copy Markdown
Contributor

@Kaniska244 Kaniska244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @daeghanelkin

Thank you for the contribution. Would you kindly bump the aws-cli feature version.

The ZSH completer script provided by the aws/aws-cli repo is not set up
to be used as a completer function, and thus cannot be dropped into a
directory like ~/.oh-my-zsh/completions and be expected to work.

This change also moves the completer to a more standard location for system-wide
ZSH completions, and adds the necessary header to make it work as a completer.
@daeghanelkin daeghanelkin force-pushed the fix/aws-cli-zsh-completion branch from 058b65c to fdea347 Compare April 21, 2026 11:42
@daeghanelkin daeghanelkin requested a review from Kaniska244 April 21, 2026 11:42
@daeghanelkin
Copy link
Copy Markdown
Author

@Kaniska244 Sorry about that, I remember reading about bumping the version and completely forgot to do it!

Copy link
Copy Markdown
Contributor

@Kaniska244 Kaniska244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be further reviewed by maintainers.

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