Skip to content

Fix aws-cli completion for ZSH#1627

Merged
abdurriq merged 1 commit intodevcontainers:mainfrom
daeghanelkin:fix/aws-cli-zsh-completion
Apr 28, 2026
Merged

Fix aws-cli completion for ZSH#1627
abdurriq merged 1 commit intodevcontainers:mainfrom
daeghanelkin:fix/aws-cli-zsh-completion

Conversation

@daeghanelkin
Copy link
Copy Markdown
Contributor

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
Contributor 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
Contributor 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.

@abdurriq abdurriq merged commit 71c999d into devcontainers:main Apr 28, 2026
11 checks passed
@daeghanelkin daeghanelkin deleted the fix/aws-cli-zsh-completion branch April 28, 2026 15:49
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.

3 participants