Skip to content

Commit

Permalink
OWNERS_ALIASES unsupported on openshift/release subdirectories (#144)
Browse files Browse the repository at this point in the history
Signed-off-by: Pierangelo Di Pilato <[email protected]>
  • Loading branch information
pierDipi authored Apr 3, 2024
1 parent 9a5fc2b commit 6ac25ce
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 38 deletions.
20 changes: 18 additions & 2 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -1,7 +1,23 @@
# The OWNERS file is used by prow to automatically merge approved PRs.

approvers:
- serverless-approvers
- alanfx
- aliok
- creydr
- lberk
- matzew
- mgencur
- pierDipi
- rhuss
- skonto
- ReToCode

reviewers:
- serverless-reviewers
- aliok
- creydr
- matzew
- mgencur
- pierDipi
- rhuss
- skonto
- ReToCode
27 changes: 0 additions & 27 deletions OWNERS_ALIASES

This file was deleted.

9 changes: 0 additions & 9 deletions pkg/prowgen/prowgen.go
Original file line number Diff line number Diff line change
Expand Up @@ -240,18 +240,9 @@ func SaveReleaseBuildConfiguration(outConfig *string, cfg ReleaseBuildConfigurat
log.Printf("failed to read file: %v", err)
return nil
}
ownerAliases, err := os.ReadFile("OWNERS_ALIASES")
if err != nil {
// Log just a warning
log.Printf("failed to read file: %v", err)
return nil
}
if err := os.WriteFile(filepath.Join(dir, "OWNERS"), owners, os.ModePerm); err != nil {
return err
}
if err := os.WriteFile(filepath.Join(dir, "OWNERS_ALIASES"), ownerAliases, os.ModePerm); err != nil {
return err
}

return nil
}
Expand Down

0 comments on commit 6ac25ce

Please sign in to comment.