Skip to content

Remove web-frameworks from dogstatsd ownership - #993

Merged
StephenWakely merged 1 commit into
masterfrom
stephen/codeowners_amp
Sep 29, 2026
Merged

StephenWakely merged 1 commit into
masterfrom
stephen/codeowners_amp

Conversation

@StephenWakely

Copy link
Copy Markdown
Contributor

Requirements for Contributing to this repository

  • Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
  • The pull request must only fix one issue, or add one feature, at the time.
  • The pull request must update the test suite to demonstrate the changed functionality.
  • After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see CONTRIBUTING.

What does this PR do?

Drops web-frameworks from ownership of the dogstatsd part of this library.

Description of the Change

Alternate Designs

Possible Drawbacks

Verification Process

Additional Notes

Release Notes

Review checklist (to be filled by reviewers)

  • Feature or bug fix MUST have appropriate tests (unit, integration, etc...)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
  • PR should not have do-not-merge/ label attached.
  • If Applicable, issue must have kind/ and severity/ labels attached at least.

@StephenWakely
StephenWakely requested a review from a team as a code owner September 25, 2026 16:40
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-25T16:41:17.799828Z f78ab54 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@StephenWakely StephenWakely added the changelog/no-changelog Changes don't appear in changelog label Sep 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The ownership change is focused, with only a minor non-blocking consistency nit.

Review effort: Lite
Findings: None

What changed in this PR

Updates .github/CODEOWNERS to remove @DataDog/web-frameworks from core DogStatsD ownership.

Changes:

  • Assigns core DogStatsD paths to @DataDog/agent-metric-pipelines.
  • Preserves existing shared ownership for other paths.
File Summary
.github/​CODEOWNERS Adjusts DogStatsD ownership assignments; a non-blocking nit remains for performance tests.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants