Skip to content

chore(codeowners): give FFE sole ownership of product paths - #4217

Draft
leoromanovsky wants to merge 1 commit into
masterfrom
leoromanovsky/ffe-sole-product-codeowners
Draft

leoromanovsky wants to merge 1 commit into
masterfrom
leoromanovsky/ffe-sole-product-codeowners

Conversation

@leoromanovsky

Copy link
Copy Markdown
Contributor

Description

Make FFE the sole owner of dedicated Feature Flags and OpenFeature product paths.

Motivation

Route review and test attribution for dedicated FFE product work to the FFE SDK team.

Changes

Remove product co-owners and add tracer/ffe.c and tracer/ffe.h.

Decisions

Keep the shared libdatadog submodule and shared tracer infrastructure co-owned.

Additional Notes

Validated effective ownership of 35 dedicated FFE paths and checked all 74,331 tracked files for unintended ownership changes; none found. git diff --check passed.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

Remove product co-owners and add `tracer/ffe.c` and `tracer/ffe.h`.

Environment: Datadog workspace
@datadog-official

datadog-official Bot commented Sep 18, 2026

Copy link
Copy Markdown

Pipelines  Tests

Unblock PR with BitsAI

❌ Errors

Your PR has failed checks. Please review the issues below and take necessary action before merging.

🚦 8 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-php | check libxml2 version — 🔧 Needs a code fix, caused by this PR

View more details · View in GitLab

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [7.3, valgrind] — 🔧 Needs a code fix, caused by this PR

View more details · View in GitLab

DataDog/apm-reliability/dd-trace-php | test_web_symfony_62: [8.3, apache2handler] — 🔧 Needs a code fix, caused by this PR

View more details · View in GitLab

View all 8 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 67.44% (-0.05%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 7233098 | Docs | View more details | Give us feedback!

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.

1 participant