Skip to content

add default CODEOWNERS - #1055

Merged
rapids-bot[bot] merged 2 commits into
rapidsai:mainfrom
jameslamb:add-default-codeowners
Jul 21, 2026
Merged

add default CODEOWNERS#1055
rapids-bot[bot] merged 2 commits into
rapidsai:mainfrom
jameslamb:add-default-codeowners

Conversation

@jameslamb

@jameslamb jameslamb commented Jul 13, 2026

Copy link
Copy Markdown
Member

Description

Contributes to rapidsai/build-planning#221

#1054 only touched .clang-format. There's no matching CODEOWNERS rule for that, so no PR reviewer was assigned.

This proposes making rapids-cmake-coedowners the default reviewers for any files not matched by other rules in .github/CODEOWNERS, to avoid that in the future.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The cmake-format.json is up to date with these changes.
  • I have added new files under rapids-cmake/
    • I have added include guards (include_guard(GLOBAL))
    • I have added the associated docs/ rst file and update the api.rst

@jameslamb
jameslamb requested a review from a team as a code owner July 13, 2026 17:34
@jameslamb
jameslamb requested a review from msarahan July 13, 2026 17:34
@jameslamb jameslamb added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jul 13, 2026
@vyasr

vyasr commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

/merge

@rapids-bot
rapids-bot Bot merged commit e0607fa into rapidsai:main Jul 21, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants