Skip to content

Commit e2230d6

Browse files
address gemini comments
1 parent 5c15009 commit e2230d6

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

cfg/optimize_test.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -84,10 +84,8 @@ func resetMetadataEndpoints(t *testing.T) {
8484

8585
// Helper function to detect if a given flag is present in the map of optimized flags.
8686
func isFlagPresentInOptimizationResults(optimizationResults map[string]OptimizationResult, flag string) bool {
87-
if _, ok := optimizationResults[flag]; ok {
88-
return true
89-
}
90-
return false
87+
_, ok := optimizationResults[flag]
88+
return ok
9189
}
9290

9391
func TestGetMachineType_Success(t *testing.T) {

cmd/root.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ of Cloud Storage FUSE, see https://cloud.google.com/storage/docs/gcs-fuse.`,
170170

171171
isSet := &pflagAsIsValueSet{fs: rootCmd.PersistentFlags()}
172172
optimizedFlags := configObj.ApplyOptimizations(isSet)
173-
optimizedFlagNames := []string{}
173+
optimizedFlagNames := make([]string, 0, len(optimizedFlags))
174174
for flag := range optimizedFlags {
175175
optimizedFlagNames = append(optimizedFlagNames, flag)
176176
}

0 commit comments

Comments
 (0)