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

Make the CMS teams less specific #110

Merged
merged 3 commits into from
Jun 1, 2023
Merged

Make the CMS teams less specific #110

merged 3 commits into from
Jun 1, 2023

Conversation

rustydb
Copy link
Contributor

@rustydb rustydb commented Jun 1, 2023

Summary and Scope

  • Fixes: Waiting on CODEOWNERS for CMS even though CMS has already approved a PR.

Issue Type

  • Docs Pull Request

CMS is broken up into 13 teams in GitHub. More often than not a qualified CMS team member will approve a PR but they won't exist in the sub-team specified in CODEOWNERS.

This change replaces the code owner for CMS items with the parent team, cray-management-systems.

Prerequisites

  • I have included documentation in my PR (or it is not required)
  • I tested this on internal system (if yes, please include results or a description of the test)
  • I tested this on a vshasta system (if yes, please include results or a description of the test)

Idempotency

Risks and Mitigations

CMS is broken up into 13 teams in GitHub. More often than not a
qualified CMS team member will approve a PR but they won't exist in the
sub-team specified in CODEOWNERS.

This change replaces the code owner for CMS items with the parent
team, cray-management-systems.
.github/CODEOWNERS Outdated Show resolved Hide resolved
.github/CODEOWNERS Outdated Show resolved Hide resolved
rustydb and others added 2 commits June 1, 2023 13:58
Co-authored-by: Mitch Harding <mitchell.harding@hpe.com>
@rustydb rustydb requested a review from mharding-hpe June 1, 2023 18:59
@rustydb rustydb merged commit 276b254 into master Jun 1, 2023
@rustydb rustydb deleted the FIX_CODEOWNERS branch June 1, 2023 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants