Skip to content

cp: Bump NVRX#2563

Closed
ko3n1g wants to merge 1 commit intor0.3.0from
ko3n1g/cp/b78eb023034d2a744dbb2c2ca5410dc600d75407-2
Closed

cp: Bump NVRX#2563
ko3n1g wants to merge 1 commit intor0.3.0from
ko3n1g/cp/b78eb023034d2a744dbb2c2ca5410dc600d75407-2

Conversation

@ko3n1g
Copy link
Copy Markdown
Contributor

@ko3n1g ko3n1g commented Feb 26, 2026

What does this PR do ?

Add a one line overview of what this PR aims to accomplish.

Changelog

  • Add specific line by line info of high level changes in this PR.

GitHub Actions CI

See the CI sectionin the Contributing doc for how to trigger the CI. A Nvidia developer will need to approve and trigger the CI for external contributors.

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?
  • Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • Reviewer: Does the PR have correct import guards for all optional libraries?

If you haven't finished some of the above items you can still open "Draft" PR.

Additional Information

  • Related to # (issue)

Summary by CodeRabbit

  • Chores
    • Removed an override configuration for a system dependency.

Signed-off-by: oliver könig <okoenig@nvidia.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d3e4812 and 82c71aa.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • pyproject.toml
💤 Files with no reviewable changes (1)
  • pyproject.toml

📝 Walkthrough

Walkthrough

A single dependency entry was removed from the pyproject.toml override-dependencies list. The nvidia-resiliency-ext package, previously installed from a specific Git source, has been deleted from the configuration.

Changes

Cohort / File(s) Summary
Dependency Configuration
pyproject.toml
Removed nvidia-resiliency-ext git-based dependency override entry from override-dependencies list.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

Run CICD, cherry-pick

Suggested reviewers

  • thomasdhc
  • erhoo82
🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and non-descriptive, using abbreviations (NVRX) without context and not clearly conveying what 'Bump' means or what the actual change accomplishes. Consider a more descriptive title such as 'Remove nvidia-resiliency-ext from override-dependencies' that clearly indicates the main change being made.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Test Results For Major Changes ✅ Passed This PR removes a pinned git source version of nvidia-resiliency-ext from override-dependencies. Since this is a minor dependency management update, the absence of test documentation does not constitute a failure per the custom check criteria.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ko3n1g/cp/b78eb023034d2a744dbb2c2ca5410dc600d75407-2

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 and usage tips.

@ko3n1g ko3n1g mentioned this pull request Feb 26, 2026
5 tasks
@ko3n1g
Copy link
Copy Markdown
Contributor Author

ko3n1g commented Feb 26, 2026

merged via #2509

@ko3n1g ko3n1g closed this Feb 26, 2026
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