Skip to content
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.

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

chore: Reorganize and refresh CODEOWNERS #10008

Merged
merged 1 commit into from
Dec 6, 2024
Merged

Conversation

ololobus
Copy link
Member

@ololobus ololobus commented Dec 4, 2024

Problem

We didn't have a codeowner for /compute, so nobody was auto-assigned for PRs like #9973

Summary of changes

While on it:

  1. Group codeowners into sections.
  2. Remove control plane from the /compute_tools because it's primarily the internal compute_ctl code.
  3. Add control plane (and compute) to /libs/compute_api because that's the shared public interface of the compute.

CODEOWNERS Show resolved Hide resolved
Copy link
Member

@bayandin bayandin left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@jcsp jcsp left a comment

Choose a reason for hiding this comment

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

Looks like nothing substantial changed for storage, LGTM

Copy link

github-actions bot commented Dec 4, 2024

7040 tests run: 6732 passed, 0 failed, 308 skipped (full report)


Flaky tests (1)

Postgres 14

Code coverage* (full report)

  • functions: 31.4% (8318 of 26471 functions)
  • lines: 47.8% (65381 of 136877 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
6b9cff8 at 2024-12-05T19:11:29.490Z :recycle:

We didn't have a codeowner for `/compute`, so nobody was auto-assigned
for PRs like #9973

While on it:
1. Group codeowners into sections.
2. Remove control plane from the `/compute_tools` because it's
   primarily the internal `compute_ctl` code.
3. Add control plane (and compute) to `/libs/compute_api` because that's
   the shared public interface of the compute.
@ololobus ololobus force-pushed the alexk/compute-codeowners branch from 196e1fa to 6b9cff8 Compare December 5, 2024 17:48
@ololobus ololobus added this pull request to the merge queue Dec 6, 2024
Merged via the queue into main with commit fa07097 Dec 6, 2024
82 checks passed
@ololobus ololobus deleted the alexk/compute-codeowners branch December 6, 2024 11:45
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.

4 participants