Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Commit

Permalink
cli/cmd: remove use of ... from applyComponents()
Browse files Browse the repository at this point in the history
As it does not add any value, as we create a slice of objects anyway
instead of specifying them by hand.

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
  • Loading branch information
invidian committed Oct 14, 2020
1 parent 4efffd2 commit df9e43b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cli/cmd/component-apply.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,14 +83,14 @@ func componentApply(contextLogger *log.Entry, componentsList []string) error {
return fmt.Errorf("suitable kubeconfig file not found. Did you run 'lokoctl cluster apply' ?")
}

if err := applyComponents(lokoConfig, kubeconfig, componentsToApply...); err != nil {
if err := applyComponents(lokoConfig, kubeconfig, componentsToApply); err != nil {
return fmt.Errorf("applying components: %w", err)
}

return nil
}

func applyComponents(lokoConfig *config.Config, kubeconfig []byte, componentNames ...string) error {
func applyComponents(lokoConfig *config.Config, kubeconfig []byte, componentNames []string) error {
for _, componentName := range componentNames {
fmt.Printf("Applying component '%s'...\n", componentName)

Expand Down

0 comments on commit df9e43b

Please sign in to comment.