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.

Sign up for GitHub

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

Jump to bottom

Update codeowners #73159

Merged
merged 3 commits into from
Sep 28, 2023
Merged

Update codeowners #73159

merged 3 commits into from
Sep 28, 2023

Conversation

Eve832
Copy link
Collaborator

@Eve832 Eve832 commented Aug 10, 2023

Hey folks; If I've tagged you as a reviewer, it's because you're on a team that I've assigned in CODEOWNERS to cover a folder that docs can't continue to own.
I hope that this doesn't greatly increase the overhead for anyone, but I've been noticing PRs going stale because they get lost in all the notifications that the docs team gets for the 15+ repos we work in. Please let me know if you don't agree with the assignations. Thanks!

In addition to updating the CODEOWNER file, I've deleted the name of a former employee from maintainers.md

@Eve832 Eve832 added type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes labels Aug 10, 2023
@grafana-delivery-bot grafana-delivery-bot bot added this to the 10.2.x milestone Aug 10, 2023
jdbaldry
jdbaldry reviewed Aug 10, 2023
View reviewed changes
.github/CODEOWNERS Outdated Show resolved Hide resolved
@Eve832 @jdbaldry
Update .github/CODEOWNERS
4228eb5 
Co-authored-by: Jack Baldry <jack.baldry@grafana.com>
@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale Issue with no recent activity label Sep 10, 2023
tolzhabayev
tolzhabayev approved these changes Sep 21, 2023
View reviewed changes
Copy link
Contributor

@tolzhabayev tolzhabayev left a comment

Choose a reason for hiding this comment

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

Probably a good idea to sync with main first before merging.

@github-actions github-actions bot removed the stale Issue with no recent activity label Sep 22, 2023
@Eve832 Eve832 enabled auto-merge (squash) September 28, 2023 16:10
@Eve832 Eve832 disabled auto-merge September 28, 2023 16:38
@Eve832 Eve832 merged commit d1a6e83 into main Sep 28, 2023
@Eve832 Eve832 deleted the cdowners-update branch September 28, 2023 18:31
@zerok zerok modified the milestones: 10.2.x, 10.2.0 Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdbaldry jdbaldry jdbaldry left review comments

@tolzhabayev tolzhabayev tolzhabayev approved these changes

@moxious moxious Awaiting requested review from moxious

@mauigrl05 mauigrl05 Awaiting requested review from mauigrl05

@RichiH RichiH Awaiting requested review from RichiH RichiH is a code owner

Assignees
No one assigned
Labels
no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project
Projects
None yet
Milestone
10.2.0
Development

Successfully merging this pull request may close these issues.
4 participants
@Eve832 @tolzhabayev @jdbaldry @zerok