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

✨ Provide a cli option to enable and disable completion command #1776

Merged
merged 1 commit into from
Nov 4, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
72 changes: 0 additions & 72 deletions cmd/completion.go

This file was deleted.

2 changes: 1 addition & 1 deletion cmd/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,9 @@ func main() {
&pluginv2.Plugin{},
),
cli.WithExtraCommands(
newCompletionCmd(),
version.NewCmd(),
),
cli.WithCompletion,
camilamacedo86 marked this conversation as resolved.
Show resolved Hide resolved
)
if err != nil {
log.Fatal(err)
Expand Down
2 changes: 1 addition & 1 deletion docs/book/src/reference/completion.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Enabling shell autocompletion
The Kubebuilder completion script for Bash can be generated with the command `kubebuilder completion bash` as the Kubebuilder completion script for Zsh can be generated with the command `kubebuilder completion zsh`.
The Kubebuilder completion script can be generated with the command `kubebuilder completion [bash|zsh|powershell]`.
Note that sourcing the completion script in your shell enables Kubebuilder autocompletion.

<aside class="note">
Expand Down
15 changes: 15 additions & 0 deletions pkg/cli/cli.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,9 @@ type cli struct {
// Whether the command is requesting help.
doGenericHelp bool

// Whether to add a completion command to the cli
completionCommand bool

// Plugins injected by options.
pluginsFromOptions map[string][]plugin.Base
// Default plugins injected by options. Only one plugin per project version
Expand Down Expand Up @@ -173,6 +176,12 @@ func WithExtraCommands(cmds ...*cobra.Command) Option {
}
}

// WithCompletion is an Option that adds the completion subcommand.
func WithCompletion(c *cli) error {
c.completionCommand = true
return nil
}

// initialize initializes the cli.
func (c *cli) initialize() error {
// Initialize cli with globally-relevant flags or flags that determine
Expand Down Expand Up @@ -345,6 +354,12 @@ func (c cli) buildRootCmd() *cobra.Command {
rootCmd.AddCommand(alphaCmd)
}

// kubebuilder completion
// Only add completion if requested
if c.completionCommand {
rootCmd.AddCommand(c.newCompletionCmd())
}

// kubebuilder create
createCmd := c.newCreateCmd()
// kubebuilder create api
Expand Down
17 changes: 17 additions & 0 deletions pkg/cli/cli_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -244,6 +244,23 @@ var _ = Describe("CLI", func() {
Expect(c.(*cli).extraCommands[0].Use).To(Equal(commandTest.Use))
})
})

Context("WithCompletion", func() {
It("should add the completion command if requested", func() {
By("not providing WithCompletion")
c, err = New(WithDefaultPlugins(pluginAV1), WithPlugins(allPlugins...))
Expect(err).NotTo(HaveOccurred())
Expect(c).NotTo(BeNil())
Expect(c.(*cli).completionCommand).To(BeFalse())

By("providing WithCompletion")
c, err = New(WithCompletion, WithDefaultPlugins(pluginAV1), WithPlugins(allPlugins...))
Expect(err).NotTo(HaveOccurred())
Expect(c).NotTo(BeNil())
Expect(c.(*cli).completionCommand).To(BeTrue())
})
})

})

})
108 changes: 108 additions & 0 deletions pkg/cli/completion.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,108 @@
/*
Copyright 2020 The Kubernetes Authors.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

package cli

import (
"fmt"
"os"

"github.com/spf13/cobra"
)

func (c *cli) newBashCmd() *cobra.Command {
return &cobra.Command{
Use: "bash",
Short: "Load bash completions",
Example: fmt.Sprintf(`# To load completion for this session, execute:
$ source <(%[1]s completion bash)

# To load completions for each session, execute once:
Linux:
$ %[1]s completion bash > /etc/bash_completion.d/%[1]s
MacOS:
$ %[1]s completion bash > /usr/local/etc/bash_completion.d/%[1]s
Adirio marked this conversation as resolved.
Show resolved Hide resolved
`, c.commandName),
RunE: func(cmd *cobra.Command, cmdArgs []string) error {
return cmd.Root().GenBashCompletion(os.Stdout)
},
}
}

func (c *cli) newZshCmd() *cobra.Command {
return &cobra.Command{
Use: "zsh",
Short: "Load zsh completions",
Example: fmt.Sprintf(`# If shell completion is not already enabled in your environment you will need
# to enable it. You can execute the following once:
$ echo "autoload -U compinit; compinit" >> ~/.zshrc

# To load completions for each session, execute once:
$ %[1]s completion zsh > "${fpath[1]}/_%[1]s"

# You will need to start a new shell for this setup to take effect.
`, c.commandName),
RunE: func(cmd *cobra.Command, cmdArgs []string) error {
return cmd.Root().GenZshCompletion(os.Stdout)
},
}
}

/* TODO: support fish code completion
At the time this comment is written, the imported spf13.cobra version does not support fish completion.
However, fish completion has been added to new spf13.cobra versions. When a new spf13.cobra version that
supports it is used, uncomment this command and add it to the base completion command.
func (c *cli) newFishCmd() *cobra.Command {
return &cobra.Command{
Use: "fish",
Short: "Load fish completions",
Example: fmt.Sprintf(`# To load completion for this session, execute:
$ %[1]s completion fish | source

# To load completions for each session, execute once:
$ %[1]s completion fish > ~/.config/fish/completions/%[1]s.fish`, c.commandName),
RunE: func(cmd *cobra.Command, cmdArgs []string) error {
return cmd.Root().GenFishCompletion(os.Stdout)
},
}
}
*/

func (c *cli) newPowerShellCmd() *cobra.Command {
return &cobra.Command{
Use: "powershell",
Short: "Load powershell completions",
RunE: func(cmd *cobra.Command, cmdArgs []string) error {
return cmd.Root().GenPowerShellCompletion(os.Stdout)
},
}
}

func (c *cli) newCompletionCmd() *cobra.Command {
cmd := &cobra.Command{
Use: "completion",
Short: "Load completions for the specified shell",
Long: fmt.Sprintf(`Output shell completion code for the specified shell.
The shell code must be evaluated to provide interactive completion of %[1]s commands.
Detailed instructions on how to do this for each shell are provided in their own commands.
`, c.commandName),
}
cmd.AddCommand(c.newBashCmd())
cmd.AddCommand(c.newZshCmd())
// cmd.AddCommand(c.newFishCmd()) // TODO: uncomment when adding fish completion
cmd.AddCommand(c.newPowerShellCmd())
Adirio marked this conversation as resolved.
Show resolved Hide resolved
return cmd
}