Skip to content

Commit 9e2105f

Browse files
committed
fix(actions): resolve reported issues in golangci-lint action
This change will fix the reported issued in the golangci-lint GitHub action related to the prealloc linter. Signed-off-by: Rob Casale <rcasale48@verizon.net>
1 parent d82c896 commit 9e2105f

File tree

6 files changed

+9
-8
lines changed

6 files changed

+9
-8
lines changed

pkg/app/app.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ func WithInteractive(interactive bool) Option {
124124
// a selection is displayed and the user must choose one
125125
func DefaultSelectCluster(discoverOutput *discovery.DiscoverOutput) (*discovery.Cluster, error) {
126126
clusterNameToID := make(map[string]string)
127-
options := []string{}
127+
options := make([]string, 0, len(discoverOutput.Clusters))
128128

129129
for _, cluster := range discoverOutput.Clusters {
130130
clusterNameToID[cluster.Name] = cluster.ID

pkg/aws/resolve.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,9 @@ func ResolveRegion(cfg config.ConfigurationSet) error {
6868
regionFilter = regionFilterCfg.Value.(string)
6969
}
7070

71-
options := []string{}
72-
options = append(options, ResolvePartitionRegions(partitionID)...)
71+
partitionList := ResolvePartitionRegions(partitionID)
72+
options := make([]string, 0, len(partitionList))
73+
options = append(options, partitionList...)
7374

7475
options, err := utils.RegexFilter(options, regionFilter)
7576
if err != nil {

pkg/azure/identity/client.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,7 @@ func formatXMLText(s string) (string, error) {
342342
func (c *AzureADClient) encodeQueryParams(params map[string]string) string {
343343
var buffer bytes.Buffer
344344

345-
keys := []string{}
345+
keys := make([]string, 0, len(params))
346346

347347
for k := range params {
348348
keys = append(keys, k)

pkg/config/configset.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ func (s *configSet) Get(name string) *Item {
160160
}
161161

162162
func (s *configSet) GetAll() []*Item {
163-
items := []*Item{}
163+
items := make([]*Item, 0, len(s.config))
164164
for _, item := range s.config {
165165
items = append(items, item)
166166
}

pkg/provider/registry/registry.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ func ListDiscoveryPluginRegistrations() []*DiscoveryPluginRegistration {
131131
pluginsLock.Lock()
132132
defer pluginsLock.Unlock()
133133

134-
plugins := []*DiscoveryPluginRegistration{}
134+
plugins := make([]*DiscoveryPluginRegistration, 0, len(discoveryPlugins))
135135
for _, registration := range discoveryPlugins {
136136
plugins = append(plugins, registration)
137137
}
@@ -143,7 +143,7 @@ func ListIdentityPluginRegistrations() []*IdentityPluginRegistration {
143143
pluginsLock.Lock()
144144
defer pluginsLock.Unlock()
145145

146-
plugins := []*IdentityPluginRegistration{}
146+
plugins := make([]*IdentityPluginRegistration, 0, len(identityPlugins))
147147
for _, registration := range identityPlugins {
148148
plugins = append(plugins, registration)
149149
}

pkg/provider/selection.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ import (
2929
type SelectItemFunc func(prompt string, items map[string]string) (string, error)
3030

3131
func DefaultItemSelection(promptMessage string, items map[string]string) (string, error) {
32-
options := []string{}
32+
options := make([]string, 0, len(items))
3333

3434
for key := range items {
3535
options = append(options, key)

0 commit comments

Comments
 (0)