Skip to content

Add github.com/KarpelesLab/minlz-rs to ports#52

Merged
klauspost merged 1 commit into
minio:mainfrom
klauspost:karplab-port
Jul 3, 2026
Merged

Add github.com/KarpelesLab/minlz-rs to ports#52
klauspost merged 1 commit into
minio:mainfrom
klauspost:karplab-port

Conversation

@klauspost

@klauspost klauspost commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Documentation
    • Updated the Ports section in the README for clearer table formatting and alignment.
    • Refreshed the Rust port entries with updated license details and repository links.
    • Standardized the C port row formatting to match the rest of the table.

@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: ec8effc7-072d-481f-a416-cf8fd899e096

📥 Commits

Reviewing files that changed from the base of the PR and between df67e49 and 680d15b.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The README.md Ports table was updated to revise license information for Rust port entries (one now listing BSD-3-Clause, another Apache 2.0) with corresponding repository link updates, plus formatting adjustments to the table header and C row alignment.

Changes

README Ports Table

Layer / File(s) Summary
Update ports table license entries and formatting
README.md
Rust port license values and repo links updated; table header and C row formatting normalized.

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

Poem

A table hopped, its rows aligned,
Rust licenses now clearly signed,
BSD here, Apache there,
C stands neat without a care,
This bunny thumps in README delight! 🐇

🚥 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 matches the main change: adding the KarpelesLab Rust port to the ports list.
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.

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.

@klauspost klauspost merged commit 05790d8 into minio:main Jul 3, 2026
59 checks passed
@klauspost klauspost deleted the karplab-port branch July 3, 2026 12:35
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