Skip to content

chore: remove old group references from CODEOWNERS - #47763

Merged
jose-ledesma merged 1 commit into
masterfrom
chore/cleanup-codeowners-groups
Jul 9, 2026
Merged

chore: remove old group references from CODEOWNERS#47763
jose-ledesma merged 1 commit into
masterfrom
chore/cleanup-codeowners-groups

Conversation

@jose-ledesma

@jose-ledesma jose-ledesma commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

This PR cleans up CODEOWNERS after the infrastructure teams have been renamed.

Removes old group references:

  • @supabase/infra → (removed, now @supabase/platform)

The new group names are now in use and working correctly.

Summary by CodeRabbit

  • Chores
    • Updated ownership settings for one shared data area to reflect the current team responsible for it.

@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
design-system Ready Ready Preview, Comment Jul 9, 2026 8:09am
docs Ready Ready Preview, Comment, Open in v0 Jul 9, 2026 8:09am
studio-self-hosted Ready Ready Preview, Comment Jul 9, 2026 8:09am
studio-staging Ready Ready Preview, Comment Jul 9, 2026 8:09am
ui-library Ready Ready Preview, Comment Jul 9, 2026 8:09am
zone-www-dot-com Ready Ready Preview, Comment, Open in v0 Jul 9, 2026 8:09am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
studio Ignored Ignored Jul 9, 2026 8:09am

Request Review

@supabase

supabase Bot commented Jul 9, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This change modifies a single line in the CODEOWNERS file, removing @supabase/infra as a code owner for /packages/shared-data/compute-disk-limits.ts while retaining @supabase/platform as the owner.

Changes

CODEOWNERS Update

Layer / File(s) Summary
Update ownership for compute-disk-limits.ts
.github/CODEOWNERS
Removes @supabase/infra from the code owner list for this file, keeping only @supabase/platform.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • supabase/supabase#47131: Both PRs edit the same CODEOWNERS rule for /packages/shared-data/compute-disk-limits.ts, toggling between @supabase/infra and @supabase/platform.

Suggested reviewers: joshenlim

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description mentions the CODEOWNERS cleanup but omits the required template sections and issue/context details. Add the template sections: CONTRIBUTING confirmation, change type, current behavior with issue links, new behavior, and any additional context.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title matches the main change: removing outdated CODEOWNERS group references.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/cleanup-codeowners-groups

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jose-ledesma
jose-ledesma enabled auto-merge (squash) July 9, 2026 08:06
@jose-ledesma
jose-ledesma disabled auto-merge July 9, 2026 08:06
@jose-ledesma
jose-ledesma enabled auto-merge (squash) July 9, 2026 08:07
@jose-ledesma
jose-ledesma merged commit c76a300 into master Jul 9, 2026
34 checks passed
@jose-ledesma
jose-ledesma deleted the chore/cleanup-codeowners-groups branch July 9, 2026 08:09
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Braintrust eval report

Assistant (master-1783584821)

Score Average Improvements Regressions
Knowledge Usage 94.9% (-5pp) - 2 🔴
SQL Identifier Quoting 100% (+0pp) - -
SQL Validity 100% (+0pp) - -
Tool Usage 76.5% (+2pp) 1 🟢 1 🔴
Time_to_first_token 3.62tok (+1.45tok) - 29 🔴
Llm_calls 2.74 (+0.16) 12 🟢 10 🔴
Tool_calls 2.91 (-0.05) 9 🟢 10 🔴
Errors 5.28 (+0) 5 🟢 4 🔴
Llm_errors 0 (+0) - -
Tool_errors 0 (+0) - -
Prompt_tokens 24251.32tok (+558.48tok) 14 🟢 11 🔴
Prompt_cached_tokens 7191.54tok (-529.66tok) 10 🟢 16 🔴
Prompt_cache_creation_tokens 0tok (+0tok) - -
Prompt_cache_creation_5m_tokens 0tok (+0tok) - -
Prompt_cache_creation_1h_tokens 0tok (+0tok) - -
Completion_tokens 613.57tok (+8.69tok) 15 🟢 14 🔴
Completion_reasoning_tokens 93.31tok (-8.63tok) 11 🟢 13 🔴
Total_tokens 24864.9tok (+567.17tok) 17 🟢 12 🔴
Estimated_cost 0$ (+0$) 12 🟢 14 🔴
Duration 18.44s (+5.56s) 1 🟢 28 🔴
Llm_duration 12.09s (+3.43s) 2 🟢 27 🔴

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.

2 participants