Skip to content

🌱 Update repo owners #2852

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
openshift-merge-bot merged 1 commit into from
Aug 7, 2026
+1 −5
Merged

🌱 Update repo owners #2852

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 1 addition & 5 deletions OWNERS_ALIASES
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
aliases:
olmv1-approvers:
- grokspawn
- joelanford
- kevinrizza
- oceanc80
- joelanford
Comment on lines 4 to +5

@coderabbitai coderabbitai Bot Aug 6, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔒 Security & Privacy | 🟠 Major | ⚡ Quick win

Remove kevinrizza from olmv1-approvers.

The PR objective lists kevinrizza among the users to remove, but OWNERS_ALIASES still lists kevinrizza on Line 4. OWNERS consumes olmv1-approvers for repository approvals, so kevinrizza remains in the approval path. Remove this entry, or update the PR objective if retaining it is intentional.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@OWNERS_ALIASES` around lines 4 - 5, Remove the kevinrizza entry from the
olmv1-approvers list in OWNERS_ALIASES so OWNERS no longer includes that user in
the repository approval path.

- pedjak
- perdasilva
Comment on lines 2 to 7
- tmshort
Expand All @@ -14,12 +13,9 @@ aliases:
- fgiudici
- grokspawn
- joelanford
- oceanc80
- OchiengEd
- perdasilva
- rashmigottipati
- tmshort
- trgeiger

@tmshort tmshort Aug 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tayler?

- pedjak

api-approvers:
Expand Down