Skip to content

Conversation

aThorp96
Copy link
Contributor

@aThorp96 aThorp96 commented Sep 12, 2025

Summary:

  • Move former Approvers to emeritus.
  • Reviewers who are no longer actively maintaining pipelines-service were left as reviewers, but I don't mind removing from reviewers if preferable (cc @xinredhat WDYT?).
  • Add pipelines IC team and SMEs to Reviwers and Approvers

Details:

  • @Roming22 moved to emeritus - no longer working on Pipelines service
  • @adambkaplan moved to emeritus - no longer maintaining tekton results
  • @enarha promoted to approver - actively maintaining tekton results
  • @aThorp96 promoted to approver - working on pipelines service, leading IC team
  • @infernus01 added to reviewers - working on pipelines service, IC
  • @ab-ghosh added to reviewers - working on pipelines service, IC

- Roming22 moved to emeritus - no longer working on Pipelines service
- adambkaplan moved to emeritus - no longer maintaining tekton results
- enarha promoted to approver - actively maintaining tekton results
- aThorp96 promoted to approver - working on pipelines service, IC
- infernus01 added to reviewers - working on pipelines service, IC
- ab-ghosh added to reviewers - working on pipelines service, IC
@openshift-ci openshift-ci bot requested review from enarha and xinredhat September 12, 2025 13:36
Copy link

openshift-ci bot commented Sep 12, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: aThorp96
Once this PR has been reviewed and has the lgtm label, please assign roming22 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link
Contributor

Code Review by Gemini

The changes in the OWNERS file align perfectly with the descriptions provided in the commit message. All promotions, additions, and moves to emeritus status are correctly reflected in the diff.

No bugs or issues were found in this change. The file structure and updates are consistent with the intended modifications.

Copy link

openshift-ci bot commented Sep 12, 2025

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the redhat-appstudio org. You can then trigger verification by writing /verify-owners in a comment.

  • adambkaplan
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.

Copy link
Contributor

Code Review by Gemini

--- a/components/pipeline-service/OWNERS
+++ b/components/pipeline-service/OWNERS
@@ -6,10 +6,8 @@
   - aThorp96
 
 reviewers:
-  - Roming22
-  - adambkaplan
   - ramessesii2
   - xinredhat
   - enarha
   - aThorp96
   - mathur07

@infernus01
Copy link
Contributor

LGTM 😁

@seanconroy2021
Copy link
Member

Hey guys, any update on getting this merged :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants