Skip to content

chore: update attributions for 0.5.1 - #663

Merged
zhengluo-nv merged 1 commit into
mainfrom
nmailhot/osrb-0.5.1-main
Aug 26, 2026
Merged

zhengluo-nv merged 1 commit into
mainfrom
nmailhot/osrb-0.5.1-main

Conversation

@nv-nmailhot

@nv-nmailhot nv-nmailhot commented Aug 19, 2026 •

Copy link
Copy Markdown
Contributor

Add missing attribution for hashbrown 0.14.5, which ships in the release binaries via config -> rust-ini -> ordered-multimap but was only attributed at 0.15.5.

Remaining line changes are trailing-whitespace fixes applied by the repo's pre-commit hook.

(cherry picked from commit ca26207)

Summary by CodeRabbit

  • Documentation
    • Added the MIT license notice for hashbrown 0.14.5.
    • Normalized existing license and attribution text by removing trailing whitespace.
    • Updated NVIDIA SDK, BSD/MIT-style, and PyTorch-related notices for consistent formatting.

Add missing attribution for hashbrown 0.14.5, which ships in the
release binaries via config -> rust-ini -> ordered-multimap but was
only attributed at 0.15.5.

Remaining line changes are trailing-whitespace fixes applied by the
repo's pre-commit hook.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit ca26207)
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 8e52a3ca-315c-40c5-8782-abdcc8b1a929

📥 Commits

Reviewing files that changed from the base of the PR and between f4660d2 and 802aa3b.

📒 Files selected for processing (1)
  • NOTICES

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.


Walkthrough

The NOTICES file adds the MIT license notice for hashbrown 0.14.5 and removes trailing whitespace from existing license and attribution text.

Changes

NOTICE maintenance

Layer / File(s) Summary
Add hashbrown license entry
NOTICES
Adds repository metadata and the full MIT license text for hashbrown 0.14.5.
Normalize existing notices
NOTICES
Removes trailing whitespace from existing license text, attribution entries, and notice markers.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 802aa

This change updates dependency attribution and formatting in NOTICES without changing product behavior; no actionable merge-blocking risk remains after normal checks and review.

Poem

I’m a rabbit with a tidy pen,
New hashbrown notices join the den.
Stray spaces hop away,
Licenses stand in array,
Clean lines greet the reader again.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: updating release attributions for version 0.5.1.
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.

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

@zhengluo-nv
zhengluo-nv enabled auto-merge (squash) August 26, 2026 17:20
@zhengluo-nv

Copy link
Copy Markdown
Contributor

/ok to test 802aa3b

@zhengluo-nv
zhengluo-nv merged commit 34fc82b into main Aug 26, 2026
85 of 88 checks passed
@zhengluo-nv
zhengluo-nv deleted the nmailhot/osrb-0.5.1-main branch August 26, 2026 17:34

This branch was previously deployed

2 inactive deployments
GITLAB — 802aa3b5 Deployed Aug 19, 2026 by nv-nmailhot via Trigger CI Pipeline #2306
automated-release — 802aa3b5 Deployed Aug 19, 2026 by copy-pr-bot[bot] via Build & upload wheels (arm64) #724
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants