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

cli/container: use github.com/moby/sys/capability for completions #5480

Merged
merged 1 commit into from
Oct 1, 2024

Commits on Oct 1, 2024

  1. cli/container: use github.com/moby/sys/capability for completions

    We used a hard-coded list of capabilities that we copied from containerd,
    but the new "capability" package allows use to have a maintained list
    of capabilities.
    
    There's likely still some improvements to be made;
    
    First of all, the capability package could provide a function to get the list
    of strings.
    
    On the completion-side, we need to consider what format is most convenient;
    currently we use the canonical name (uppercase and "CAP_" prefix), however,
    tab-completion is case-sensitive by default, so requires the user to type
    uppercase letters to filter the list of options.
    
    Bash completion provides a `completion-ignore-case on` option to make completion
    case-insensitive (https://askubuntu.com/a/87066), but it looks to be a global
    option; the current cobra.CompletionOptions also don't provide this as an option
    to be used in the generated completion-script.
    
    Fish completion has `smartcase` (by default?) which matches any case if
    all of the input is lowercase.
    
    Zsh does not have a dedicated option, but allows setting matching-rules
    (see https://superuser.com/a/1092328).
    
    Signed-off-by: Sebastiaan van Stijn <[email protected]>
    thaJeztah committed Oct 1, 2024
    Configuration menu
    Copy the full SHA
    462e082 View commit details
    Browse the repository at this point in the history