Skip to content

chore: add NAT reviewers for docs#384

Merged
rapids-bot[bot] merged 1 commit into
NVIDIA:release/0.5from
willkill07:chore/relay-444-codeowners-doc-reviewers
Jul 8, 2026
Merged

chore: add NAT reviewers for docs#384
rapids-bot[bot] merged 1 commit into
NVIDIA:release/0.5from
willkill07:chore/relay-444-codeowners-doc-reviewers

Conversation

@willkill07

@willkill07 willkill07 commented Jul 8, 2026

Copy link
Copy Markdown
Member

Overview

Update documentation CODEOWNERS entries so NAT developers are requested alongside technical writers.

  • I confirm this contribution is my own work, or I have the right to submit it under this project's license.
  • I searched existing issues and open pull requests, and this does not duplicate existing work.

Details

  • Add @nvidia/NAT-developers to the docs/ and fern/ ownership rules.
  • Preserve the existing technical-writer reviewer on both rules.

Where should the reviewer start?

Review .github/CODEOWNERS; it is the only changed file.

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • Closes RELAY-444

Summary by CodeRabbit

  • Chores
    • Updated documentation ownership rules for repo docs-related paths.
    • Added an additional owner for docs/ and fern/ to broaden review coverage.

Signed-off-by: Will Killian <wkillian@nvidia.com>
@willkill07 willkill07 requested a review from a team as a code owner July 8, 2026 14:03
@github-actions github-actions Bot added size:XS PR is extra small Maintenance CI or Build or general repository maintenance labels Jul 8, 2026
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: a9c8019c-6120-4484-978a-1afe1a10c6a4

📥 Commits

Reviewing files that changed from the base of the PR and between dd55c28 and e591fca.

📒 Files selected for processing (1)
  • .github/CODEOWNERS
📜 Recent review details
⏰ Context from checks skipped due to timeout. (2)
  • GitHub Check: Check / Run
  • GitHub Check: Preview docs
🧰 Additional context used
📓 Path-based instructions (1)
{.github/**,.gitlab-ci.yml,.pre-commit-config.yaml,justfile,scripts/**}

⚙️ CodeRabbit configuration file

{.github/**,.gitlab-ci.yml,.pre-commit-config.yaml,justfile,scripts/**}: Review automation changes for reproducibility, pinned versions where appropriate, secret handling, and consistency with the documented validation matrix.
Pay attention to commands that need generated native artifacts, FFI libraries, or platform-specific environment variables.

Files:

  • .github/CODEOWNERS
🔇 Additional comments (1)
.github/CODEOWNERS (1)

10-12: LGTM!


Walkthrough

The CODEOWNERS file was updated to add @nvidia/NAT-developers as an additional owner for docs/ and fern/ paths, alongside the existing @lvojtku owner, with the associated comment updated to reflect the expanded scope.

Changes

CODEOWNERS Update

Layer / File(s) Summary
Add NAT-developers as documentation code owner
.github/CODEOWNERS
docs/ and fern/ entries now list @nvidia/NAT-developers alongside @lvojtku, with the comment updated to reflect expanded ownership.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title uses Conventional Commits and accurately summarizes the CODEOWNERS update for NAT reviewers on docs paths.
Description check ✅ Passed The description includes the required overview, details, reviewer start point, checklist items, and a related issue reference.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@willkill07 willkill07 self-assigned this Jul 8, 2026
@willkill07 willkill07 added this to the 0.5 milestone Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

@willkill07

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot rapids-bot Bot merged commit 0486d88 into NVIDIA:release/0.5 Jul 8, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance CI or Build or general repository maintenance size:XS PR is extra small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants