Skip to content

Update .cncf-maintainers & OWNERS_ALIASES#6041

Merged
jmrodri merged 1 commit into
operator-framework:masterfrom
jmrodri:fix-sanity
Sep 26, 2022
Merged

Update .cncf-maintainers & OWNERS_ALIASES#6041
jmrodri merged 1 commit into
operator-framework:masterfrom
jmrodri:fix-sanity

Conversation

@jmrodri

@jmrodri jmrodri commented Sep 26, 2022

Copy link
Copy Markdown
Member

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 jesusr@redhat.com

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

everettraven commented Sep 26, 2022

Copy link
Copy Markdown
Contributor

@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 <jmrodri@gmail.com>
@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

@everettraven everettraven left a comment

Copy link
Copy Markdown
Contributor

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 <jmrodri@gmail.com>
Signed-off-by: Daniel Campos Olivares <daniel.campos.olivares@sap.com>
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