Skip to content

Add modelopt-recipes-codeowners#1145

Open
kevalmorabia97 wants to merge 1 commit intomainfrom
kevalmorabia97-patch-1
Open

Add modelopt-recipes-codeowners#1145
kevalmorabia97 wants to merge 1 commit intomainfrom
kevalmorabia97-patch-1

Conversation

@kevalmorabia97
Copy link
Copy Markdown
Collaborator

@kevalmorabia97 kevalmorabia97 commented Mar 31, 2026

As title

Summary by CodeRabbit

Release Notes

This release contains internal maintenance updates with no user-facing changes. Code ownership configurations have been updated to improve project governance and development workflows.

Signed-off-by: Keval Morabia <28916987+kevalmorabia97@users.noreply.github.com>
@kevalmorabia97 kevalmorabia97 requested a review from a team as a code owner March 31, 2026 07:20
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: cd93ef86-c8e6-4c3b-8041-b878955bc4eb

📥 Commits

Reviewing files that changed from the base of the PR and between 74a8694 and 38aa3ee.

📒 Files selected for processing (1)
  • .github/CODEOWNERS

📝 Walkthrough

Walkthrough

The pull request adds a new CODEOWNERS entry assigning the modelopt_recipes path to the @NVIDIA/modelopt-recipes-codeowners team, extending code ownership coverage within the repository.

Changes

Cohort / File(s) Summary
CODEOWNERS Configuration
.github/CODEOWNERS
Added ownership assignment for the modelopt_recipes path to the designated codeowners team.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Add modelopt-recipes-codeowners' directly and accurately describes the main change: adding a new CODEOWNERS entry for the modelopt_recipes path.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Security Anti-Patterns ✅ Passed PR modifies only .github/CODEOWNERS file with no Python code changes, placing it outside scope of Python security review.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch kevalmorabia97-patch-1

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

@kevalmorabia97 kevalmorabia97 enabled auto-merge (squash) March 31, 2026 07:21
@github-actions
Copy link
Copy Markdown
Contributor

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://NVIDIA.github.io/Model-Optimizer/pr-preview/pr-1145/

Built to branch gh-pages at 2026-03-31 07:25 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.19%. Comparing base (74a8694) to head (38aa3ee).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1145      +/-   ##
==========================================
- Coverage   70.21%   70.19%   -0.03%     
==========================================
  Files         230      230              
  Lines       26073    26073              
==========================================
- Hits        18308    18302       -6     
- Misses       7765     7771       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant