Skip to content

Commit

Permalink
mungegithub: do not set empty assignees is none set
Browse files Browse the repository at this point in the history
Some OWNERS files might have no assignees, and that's fine.
  • Loading branch information
eparis committed Mar 24, 2016
1 parent f913e22 commit e862b7e
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions mungegithub/features/repo-updates.go
Original file line number Diff line number Diff line change
Expand Up @@ -94,11 +94,16 @@ func (o *RepoInfo) walkFunc(path string, info os.FileInfo, err error) error {

path, err = filepath.Rel(o.kubernetesDir, path)
if err != nil {
glog.Errorf("Unable to find relative path between %q and %q: %v", o.kubernetesDir, path, err)
return err
}
path = filepath.Dir(path)
o.assignees[path] = sets.NewString(c.Assignees...)
//o.owners[path] = sets.NewString(c.Assignees...)
if len(c.Assignees) > 0 {
o.assignees[path] = sets.NewString(c.Assignees...)
}
//if len(c.Owners) > 0 {
//o.owners[path] = sets.NewString(c.Owners...)
//}
return nil
}

Expand Down

0 comments on commit e862b7e

Please sign in to comment.