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 PR review strategy #5587

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Update PR review strategy #5587

merged 1 commit into from
Jun 5, 2024

Conversation

bmesuere
Copy link
Member

@bmesuere bmesuere commented Jun 5, 2024

This pull request changes the review strategy from the review team to just me. For bigger PRs, we will manually add @chvp, @niknetniko or @rien if they are available.

@bmesuere bmesuere requested a review from a team as a code owner June 5, 2024 10:48
@bmesuere bmesuere requested review from niknetniko and jorg-vr and removed request for a team June 5, 2024 10:48
@bmesuere bmesuere added the chore Repository/build/dependency maintenance label Jun 5, 2024
Copy link
Member

@niknetniko niknetniko left a comment

Choose a reason for hiding this comment

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

I'm not against reviewing small PR's, but as this will need change anyway at the end of august, fine for me :)

@bmesuere bmesuere merged commit eac9c72 into main Jun 5, 2024
15 checks passed
@bmesuere bmesuere deleted the chore/update-reviewers branch June 5, 2024 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Repository/build/dependency maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants