Skip to content

Fix MiniExcel NuGet dependency version - #4

Merged
shps951023 merged 1 commit into
mainfrom
fix/nuget-dependency-version
Sep 9, 2026
Merged

Fix MiniExcel NuGet dependency version#4
shps951023 merged 1 commit into
mainfrom
fix/nuget-dependency-version

Conversation

@shps951023

@shps951023 shps951023 commented Sep 9, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the NuGet Gallery-incompatible upper bound 2.0.0-0 with an exact MiniExcel v1 dependency [1.46.0]
  • update package verification to require the exact dependency for both target frameworks
  • synchronize the dependency statement across all maintained README translations

Validation

  • built and consumed MiniExcel.Rust 0.1.0-preview.1 from an isolated local feed
  • verified all eight native RID assets
  • inspected the generated nuspec: both dependency groups contain MiniExcel [1.46.0]
  • git diff --check

Summary by CodeRabbit

  • Documentation

    • Updated .NET package documentation across supported languages to specify MiniExcel v1 version 1.46.0.
  • Bug Fixes

    • The .NET package now uses only the exact MiniExcel version 1.46.0, preventing resolution of later 1.x releases.
    • Package verification now confirms the fixed dependency version.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 6d48f991-b4ac-4e07-8b05-7d0c0a30868f

📥 Commits

Reviewing files that changed from the base of the PR and between 18fb77a and f6bb544.

📒 Files selected for processing (13)
  • README.es.md
  • README.fr.md
  • README.ja.md
  • README.md
  • README.zh-CN.md
  • README.zh-TW.md
  • docs/i18n/README.es.md
  • docs/i18n/README.fr.md
  • docs/i18n/README.ja.md
  • docs/i18n/README.zh-CN.md
  • docs/i18n/README.zh-TW.md
  • dotnet/src/MiniExcel.Rust/MiniExcel.Rust.csproj
  • scripts/dotnet/Verify-Package.ps1

📝 Walkthrough

Walkthrough

The pull request pins the MiniExcel.Rust NuGet dependency to MiniExcel 1.46.0. Package verification and English and localized README files now use the same exact-version requirement.

Changes

MiniExcel dependency pinning

Layer / File(s) Summary
Package constraint and validation
dotnet/src/MiniExcel.Rust/MiniExcel.Rust.csproj, scripts/dotnet/Verify-Package.ps1
The project and package verification now require the exact version range [1.46.0].
Localized dependency documentation
README.md, README.*.md, docs/i18n/README.*.md
The documentation now states that MiniExcel.Rust depends exactly on MiniExcel 1.46.0.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/nuget-dependency-version

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

@shps951023
shps951023 merged commit 90f7d9f into main Sep 9, 2026
35 of 36 checks passed
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