Skip to content

Remove Docs Team as CODEOWNERS #3328

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
Merged
merged 5 commits into from
Sep 26, 2025
Merged

Remove Docs Team as CODEOWNERS #3328

merged 5 commits into from
Sep 26, 2025

Conversation

drichards-87
Copy link
Contributor

What does this PR do?

Removes Docs as CODEOWNERS from the repo. As part of a project to reduce courtesy reviews, the Docs Team is limiting CODEOWNERS entries to repos that single-source content into the docs site or where our review is critical. We’re still happy to review content if you reach out to us directly.

Additional Notes

Review checklist

Please check relevant items below:

  • This PR includes all newly recorded cassettes for any modified tests.

  • This PR does not rely on API client schema changes.

    • The CI should be fully passing.

  • Or, this PR relies on API schema changes and this is a Draft PR to include tests for that new functionality.

    • Note: CI shouldn't be run on this Draft PR, as its expected to fail without the corresponding schema changes.

@drichards-87
Remove Docs Team as CODEOWNERS
e980514 
Removes Docs as CODEOWNERS from the repo. As part of a project to reduce courtesy reviews, the Docs Team is limiting CODEOWNERS entries to repos that single-source content into the docs site or where our review is critical. We’re still happy to review content if you reach out to us directly.
@drichards-87 drichards-87 requested review from a team as code owners September 3, 2025 22:36
@drichards-87 drichards-87 added the changelog/no-changelog Changes don't appear in changelog label Sep 3, 2025
@drichards-87
Copy link
Contributor Author

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Sep 12, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-09-12 16:07:00 UTC ℹ️ Start processing command /merge

2025-09-12 16:07:07 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-09-12 17:15:11 UTC ⚠️ MergeQueue: This merge request was unqueued

[email protected] unqueued this merge request

@drichards-87
Copy link
Contributor Author

/remove

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Sep 12, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-09-12 17:15:03 UTC ℹ️ Start processing command /remove

2025-09-12 17:15:08 UTC ℹ️ Devflow: /remove

@drichards-87
Copy link
Contributor Author

/merge -m squash

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Sep 12, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-09-12 17:15:27 UTC ℹ️ Start processing command /merge -m squash

2025-09-12 17:15:42 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-09-12 18:22:36 UTC ⚠️ MergeQueue: This merge request was unqueued

[email protected] unqueued this merge request

@drichards-87
Copy link
Contributor Author

/remove

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Sep 12, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-09-12 18:22:30 UTC ℹ️ Start processing command /remove

2025-09-12 18:22:34 UTC ℹ️ Devflow: /remove

@skarimo skarimo merged commit 80cb0b3 into master Sep 26, 2025
12 checks passed
@skarimo skarimo deleted the drichards-87/remove-docs-team-codeowners branch September 26, 2025 17:01
github-actions bot pushed a commit that referenced this pull request Sep 26, 2025
@skarimo
Remove Docs Team as CODEOWNERS (#3328)
82c6e50 
Removes Docs as CODEOWNERS from the repo. As part of a project to reduce courtesy reviews, the Docs Team is limiting CODEOWNERS entries to repos that single-source content into the docs site or where our review is critical. We’re still happy to review content if you reach out to us directly. 80cb0b3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@armcburney armcburney armcburney approved these changes

Assignees
No one assigned
Labels
changelog/no-changelog Changes don't appear in changelog mergequeue-status: removed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@drichards-87 @armcburney @skarimo