Skip to content

Update OWNERS #27

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update OWNERS #27

wants to merge 1 commit into from

Conversation

@cblecker
Copy link
Member

@cblecker cblecker commented Nov 20, 2021

Add @cblecker to owners and remove Rick

@cblecker
Copy link
Member Author

cblecker commented Nov 20, 2021

/assign NautiluX

@openshift-ci openshift-ci bot requested review from RiRa12621 and jan--f November 20, 2021 22:19
RiRa12621
RiRa12621 suggested changes Nov 20, 2021
View reviewed changes
Copy link

@RiRa12621 RiRa12621 left a comment

Choose a reason for hiding this comment

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

Why would you remove the emeritus section

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 20, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: cblecker
To complete the pull request process, please ask for approval from nautilux after the PR has been reviewed.

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

paulfantom
paulfantom suggested changes Nov 22, 2021
View reviewed changes
Copy link
Contributor

@paulfantom paulfantom left a comment

Choose a reason for hiding this comment

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

I may not be working at Red Hat anymore, but I wonder why you are removing me from being an emeritus maintainer?

@cblecker
Copy link
Member Author

cblecker commented Nov 22, 2021

@RiRa12621 @paulfantom We don't have a definition of an "emeritus maintainer" for this repo/project. It shouldn't have been added in the first place.

@openshift openshift locked and limited conversation to collaborators Nov 22, 2021
@openshift openshift deleted a comment from RiRa12621 Nov 22, 2021
@openshift openshift deleted a comment from paulfantom Nov 22, 2021
@NautiluX
Copy link
Contributor

NautiluX commented Dec 22, 2021

/assign @cblecker

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jan--f jan--f Awaiting requested review from jan--f

2 more reviewers

@paulfantom paulfantom paulfantom requested changes

@RiRa12621 RiRa12621 RiRa12621 requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@cblecker cblecker

@NautiluX NautiluX

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@cblecker @NautiluX @paulfantom @RiRa12621