Skip to content

[codeowners] Remove individuals from ownership of content-service-api#13023

Merged
roboquat merged 1 commit into
mainfrom
mp/content-service-codeowners
Sep 16, 2022
Merged

[codeowners] Remove individuals from ownership of content-service-api#13023
roboquat merged 1 commit into
mainfrom
mp/content-service-codeowners

Conversation

@easyCZ

@easyCZ easyCZ commented Sep 16, 2022

Copy link
Copy Markdown
Member

Description

This PR removes individuals from codeowners of the content service. Depending on individuals is suboptimal as it introduces a bus factor when individuals are unavailable or on holidays.

This removes @geropl, @corneliusludmann and @csweichel and instead makes Team Workspace the owner.

Related Issue(s)

Fixes #

How to test

Release Notes

NONE

Documentation

Werft options:

  • /werft with-preview

@easyCZ
easyCZ requested review from a team, corneliusludmann, csweichel and geropl September 16, 2022 07:47
@github-actions github-actions Bot added the team: workspace Issue belongs to the Workspace team label Sep 16, 2022

@geropl geropl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍 for teams instead of individuals, and I don't feel I can add much value here.

@roboquat
roboquat merged commit bcdda3a into main Sep 16, 2022
@roboquat
roboquat deleted the mp/content-service-codeowners branch September 16, 2022 14:32
@roboquat roboquat added deployed: workspace Workspace team change is running in production deployed Change is completely running in production labels Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployed: workspace Workspace team change is running in production deployed Change is completely running in production release-note-none size/XS team: workspace Issue belongs to the Workspace team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants