Skip to content

chore(repo): update ownersfile #1406

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

Jump to bottom
Merged
merged 1 commit into from
Sep 11, 2025
Merged

chore(repo): update ownersfile #1406

merged 1 commit into from
Sep 11, 2025

Conversation

@christoph-jerolimov
Copy link
Member

@christoph-jerolimov christoph-jerolimov commented Sep 4, 2025

This PR

  1. replaces mostly individuals with teams, when we have a responsible rhdh team for a workspace.
  2. adds all workspaces here, incl the gaps that we have at the moment.
  3. I kept or updated some items where the UI team has define a point of contact for a workspace so that the GitHub team is specified, but also this POC (point of contact). I hope its possible that the GitHub notifications goes then just to these individuals and not the complete team...
  4. And finally I've added @caugello @CryptoRodeo for the workspaces quay, redhat-argocd and tekton.

I've done my best to kept the lists accurate. Let me know or raise another PR if something was incorrect. 😏

@christoph-jerolimov christoph-jerolimov self-assigned this Sep 4, 2025
@christoph-jerolimov christoph-jerolimov force-pushed the update-owners branch 2 times, most recently from 7acde80 to b6869ec Compare September 4, 2025 10:55
karthikjeeyar
karthikjeeyar reviewed Sep 5, 2025
View reviewed changes
Copy link
Member

@karthikjeeyar karthikjeeyar left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Sep 5, 2025
@christoph-jerolimov
chore(repo): update ownersfile
549ed5c 
Signed-off-by: Christoph Jerolimov <[email protected]>
@openshift-ci openshift-ci bot removed the lgtm label Sep 5, 2025
@openshift-ci
Copy link

openshift-ci bot commented Sep 5, 2025

New changes are detected. LGTM label has been removed.

kadel
kadel reviewed Sep 5, 2025
View reviewed changes
rohitkrai03
rohitkrai03 approved these changes Sep 5, 2025
View reviewed changes
Copy link

@rohitkrai03 rohitkrai03 left a comment

Choose a reason for hiding this comment

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

/approve

@github-actions github-actions bot added the non-workspace-changes PR changes files outside workspace directories label Sep 9, 2025
@davidfestal
Copy link
Member

davidfestal commented Sep 11, 2025

/publish

@github-actions
Copy link
Contributor

github-actions bot commented Sep 11, 2025

PR action (/publish) cancelled: PR doesn't touch only 1 workspace.

@davidfestal davidfestal merged commit 5a8a9c4 into redhat-developer:main Sep 11, 2025
1 check passed
@christoph-jerolimov christoph-jerolimov deleted the update-owners branch September 11, 2025 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kadel kadel kadel approved these changes

@davidfestal davidfestal davidfestal approved these changes

@rohitkrai03 rohitkrai03 rohitkrai03 approved these changes

@schultzp2020 schultzp2020 schultzp2020 approved these changes

@BethGriggs BethGriggs Awaiting requested review from BethGriggs

@gashcrumb gashcrumb Awaiting requested review from gashcrumb gashcrumb is a code owner

+1 more reviewer

@karthikjeeyar karthikjeeyar karthikjeeyar approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@christoph-jerolimov christoph-jerolimov

@karthikjeeyar karthikjeeyar

Labels

non-workspace-changes PR changes files outside workspace directories

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@christoph-jerolimov @davidfestal @kadel @rohitkrai03 @karthikjeeyar @schultzp2020