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

Document the 'reviewer' role #3231

Merged
merged 2 commits into from
Aug 16, 2022
Merged

Document the 'reviewer' role #3231

merged 2 commits into from
Aug 16, 2022

Conversation

negz
Copy link
Member

@negz negz commented Aug 16, 2022

Description of your changes

Pertains to #3046

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

I've done a brief proof-read.

@negz negz requested a review from a team as a code owner August 16, 2022 00:57
@negz negz requested review from bassam and jbw976 August 16, 2022 00:57
GOVERNANCE.md Outdated
Comment on lines 351 to 352
cases this can also occur by a vote of the maintainers per the voting process
below.
Copy link
Member

Choose a reason for hiding this comment

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

Might be worth linking to the conflict resolution section, I was expecting it would come in the subsequent section when reading here.

Copy link
Member

@muvaf muvaf left a comment

Choose a reason for hiding this comment

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

LGTM!

@negz
Copy link
Member Author

negz commented Aug 16, 2022

@bassam @jbw976 I need an approval from one of you on this one since you're now code owners for all governance and charter related documents.

@negz
Copy link
Member Author

negz commented Aug 16, 2022

Thanks!

@negz negz merged commit 253bd4f into crossplane:master Aug 16, 2022
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.

4 participants