Skip to content

Add CODEOWNERS: enterprise + dx teams#4

Merged
mjdavidson merged 1 commit into
mainfrom
add-codeowners
Jul 16, 2026
Merged

Add CODEOWNERS: enterprise + dx teams#4
mjdavidson merged 1 commit into
mainfrom
add-codeowners

Conversation

@mjdavidson

@mjdavidson mjdavidson commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Required by the public-repo release checks (per the release runbook): every public repo needs a CODEOWNERS file naming GitHub teams, and main must require a code-owner review.

  • * @workos/enterprise @workos/dx — domain team (SSO, DSync) plus DX, mirroring the two-owner pattern in workos/workos-node. Both teams have been granted write access so ownership resolves.
  • Paired change (applied on the ruleset side): Main branch protection now sets require_code_owner_review: true.

This is the last release-check blocker before the visibility flip.


Devin shepherd: https://app.devin.ai/sessions/4fb79d36092a40138504c5b40463c36a

@devin-ai-integration devin-ai-integration Bot 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@greptile-apps

greptile-apps Bot commented Jul 16, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds a .github/CODEOWNERS file as required by the public-repo release checklist, assigning all files to the @workos/enterprise and @workos/dx teams. No logic changes are included; the diff is a single configuration line.

  • * @workos/enterprise @workos/dx — a catch-all glob that covers every file in the repository, placed in the canonical .github/CODEOWNERS location. Syntax is valid and ordering is irrelevant with only one entry.
  • Both teams are noted in the PR description as having write access, which is required for GitHub to resolve CODEOWNERS ownership correctly.

Confidence Score: 5/5

Safe to merge — adds a single CODEOWNERS line with no runtime impact.

The change is one line of GitHub configuration: a catch-all glob assigning all files to two named teams. The syntax is correct, the file is in the right location, and both teams are described as having the write access GitHub requires to resolve ownership. There is nothing that could break at runtime or affect the application itself.

No files require special attention.

Important Files Changed

Filename Overview
.github/CODEOWNERS Adds a single catch-all CODEOWNERS rule assigning all files to @workos/enterprise and @workos/dx; syntax and placement are correct

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Pull Request opened] --> B{Does PR touch any file?}
    B -- Yes --> C[GitHub checks CODEOWNERS]
    C --> D["* matches all files"]
    D --> E["@workos/enterprise notified"]
    D --> F["@workos/dx notified"]
    E --> G{Both teams have write access?}
    F --> G
    G -- Yes --> H[Ownership resolves — review request sent]
    G -- No --> I[Ownership fails to resolve]
    H --> J{branch protection: require_code_owner_review}
    J -- true --> K[PR blocked until a code owner approves]
    K --> L[Merge allowed]
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[Pull Request opened] --> B{Does PR touch any file?}
    B -- Yes --> C[GitHub checks CODEOWNERS]
    C --> D["* matches all files"]
    D --> E["@workos/enterprise notified"]
    D --> F["@workos/dx notified"]
    E --> G{Both teams have write access?}
    F --> G
    G -- Yes --> H[Ownership resolves — review request sent]
    G -- No --> I[Ownership fails to resolve]
    H --> J{branch protection: require_code_owner_review}
    J -- true --> K[PR blocked until a code owner approves]
    K --> L[Merge allowed]
Loading

Reviews (1): Last reviewed commit: "Add CODEOWNERS: enterprise + dx teams" | Re-trigger Greptile

@devin-ai-integration

Copy link
Copy Markdown

Thanks @greptile-apps — your summary matches the intent exactly: a single catch-all rule in .github/CODEOWNERS (a canonical location alongside root and docs/), valid syntax, and both @workos/enterprise and @workos/dx have write access so ownership resolves. No runtime impact. CI is green; proceeding to merge via the Aviator queue.

@greptile-apps

greptile-apps Bot commented Jul 16, 2026

Copy link
Copy Markdown

Sounds good — thanks for confirming. Happy merging!

@mjdavidson
mjdavidson merged commit c5d0210 into main Jul 16, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant