Skip to content

Remove Docs Team as CODEOWNERS #2648

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

Remove Docs Team as CODEOWNERS #2648

merged 4 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
70664c0 
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:33
@drichards-87
Copy link
Contributor Author

/merge

@dd-devflow-routing-codex
Copy link

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

View all feedbacks in Devflow UI.

2025-09-05 18:30:10 UTC ℹ️ Start processing command /merge

2025-09-05 18:30:24 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-05 18:52:28 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 5, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-09-05 18:52:22 UTC ℹ️ Start processing command /remove

2025-09-05 18:52:25 UTC ℹ️ Devflow: /remove

@drichards-87
Copy link
Contributor Author

@NaOHman Thanks for approving this PR. I tried adding it to the merge queue, but it failed since it still needs approval from the Web Frameworks team (your approval shows up under the Web Frameworks Approvers group). Would you be able to merge it directly for me?

@drichards-87
Copy link
Contributor Author

/merge

@dd-devflow-routing-codex
Copy link

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

View all feedbacks in Devflow UI.

2025-09-09 21:33:58 UTC ℹ️ Start processing command /merge

2025-09-09 21:34:05 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 0s (p90).

2025-09-09 22:02:08 UTC ⚠️ MergeQueue: This merge request build was cancelled

[email protected] cancelled this merge request build

@drichards-87
Copy link
Contributor Author

/remove

@dd-devflow-routing-codex
Copy link

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

View all feedbacks in Devflow UI.

2025-09-09 22:02:02 UTC ℹ️ Start processing command /remove

2025-09-09 22:02:05 UTC ℹ️ Devflow: /remove

@skarimo skarimo merged commit 3635f82 into master Sep 26, 2025
15 checks passed
@skarimo skarimo deleted the drichards-87/remove-docs-team-codeowners branch September 26, 2025 17:02
github-actions bot pushed a commit that referenced this pull request Sep 26, 2025
@skarimo
Remove Docs Team as CODEOWNERS (#2648)
190cd86 
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. 3635f82
github-actions bot pushed a commit to ConnectionMaster/datadog-api-client-ruby that referenced this pull request Sep 27, 2025
@pull
Remove Docs Team as CODEOWNERS (DataDog#2648)
e5bc303 
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. 3635f82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NaOHman NaOHman NaOHman approved these changes

@armcburney armcburney armcburney approved these changes

Assignees
No one assigned
Labels
changelog/no-changelog mergequeue-status: removed
Projects
None yet
Milestone
No milestone
Development

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