Skip to content

add adysenrothman as approver#2570

Open
adysenrothman wants to merge 1 commit intokubeflow:mainfrom
adysenrothman:approver
Open

add adysenrothman as approver#2570
adysenrothman wants to merge 1 commit intokubeflow:mainfrom
adysenrothman:approver

Conversation

@adysenrothman
Copy link
Copy Markdown
Contributor

Based on the requirements, I'd like to be an approver for model registry:

✅ Met the responsibilities of the Reviewer role of the codebase for at least 3 months.

✅ Primary reviewer for at least 10 substantial PRs to the codebase
https://github.com/kubeflow/model-registry/pulls?q=is%3Apr+is%3Aclosed+assignee%3Aadysenrothman

✅ Reviewed or merged at least 30 PRs to the codebase
https://github.com/kubeflow/model-registry/pulls?q=is%3Apr+author%3Aadysenrothman+is%3Aclosed+

Signed-off-by: Adysen Rothman <85646824+adysenrothman@users.noreply.github.com>
@google-oss-prow google-oss-prow bot requested review from chambridge and fege April 10, 2026 17:25
@google-oss-prow
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

Details 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

@jonburdo
Copy link
Copy Markdown
Member

+1
@adysenrothman has a long list of great contributions to model catalog and other parts of the project!

Copy link
Copy Markdown
Member

@tarilabs tarilabs left a comment

Choose a reason for hiding this comment

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

/lgtm

Congrats @adysenrothman for your contributions !!

Copy link
Copy Markdown
Contributor

@manaswinidas manaswinidas left a comment

Choose a reason for hiding this comment

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

/lgtm

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.

4 participants