Skip to content

Commit

Permalink
machine-types: remove duplicate dedup
Browse files Browse the repository at this point in the history
PR kubernetes#6058 addded dedup in a better way than I had previously done, so
remove my more complicated and now superfluous second dedup.
  • Loading branch information
justinsb committed Nov 28, 2018
1 parent 741510a commit b78e431
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions hack/machine_types/machine_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -262,14 +262,7 @@ func run() error {

for _, f := range sortedFamilies {
output = output + fmt.Sprintf("\n// %s family", f)
previousMachine := ""
for _, m := range machines {
// Ignore duplicates
if m.Name == previousMachine {
continue
}
previousMachine = m.Name

if family := strings.Split(m.Name, ".")[0]; family == f {
var ecu string
if m.Burstable {
Expand Down

0 comments on commit b78e431

Please sign in to comment.