Skip to content
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.

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

Update .cncf-maintainers & OWNERS_ALIASES #6041

Merged
merged 1 commit into from
Sep 26, 2022

Conversation

jmrodri
Copy link
Member

@jmrodri jmrodri commented Sep 26, 2022

Description of the change:

  • removed camila and marc from cncf-maintainers
  • removed marc from OWNERS_ALIASES
  • added everettraven to OWNERS_ALIASES as reviewer

Signed-off-by: jesus m. rodriguez [email protected]

Motivation for the change:
Failing test-sanity task.

Checklist

If the pull request includes user-facing changes, extra documentation is required:

@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 17:29 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 17:29 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 17:29 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 17:29 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 17:29 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 17:29 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 17:29 Inactive
@everettraven
Copy link
Contributor

everettraven commented Sep 26, 2022

@jmrodri Looks like sanity check is now failing due to you adding me as a reviewer to OWNER_ALIASES but not .cncf_maintainers 😄

* removed camila and marc from cncf-maintainers
* removed marc from OWNERS_ALIASES
* added everettraven to OWNERS_ALIASES as reviewer

Signed-off-by: jesus m. rodriguez <[email protected]>
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 18:01 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 18:01 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 18:01 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 18:01 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 18:01 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 18:01 Inactive
@jmrodri jmrodri temporarily deployed to deploy September 26, 2022 18:01 Inactive
Copy link
Contributor

@everettraven everettraven 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 Indicates that a PR is ready to be merged. label Sep 26, 2022
@jmrodri jmrodri merged commit 8fc9626 into operator-framework:master Sep 26, 2022
dacamposol pushed a commit to dacamposol/operator-sdk that referenced this pull request Oct 6, 2022
* removed camila and marc from cncf-maintainers
* removed marc from OWNERS_ALIASES
* added everettraven to OWNERS_ALIASES as reviewer

Signed-off-by: jesus m. rodriguez <[email protected]>
Signed-off-by: Daniel Campos Olivares <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants