Skip to content

chore: update binding files and minor tweaks#279

Merged
Brooooooklyn merged 1 commit intomainfrom
10-30-chore_update_binding_files_and_minor_tweaks
Oct 30, 2025
Merged

chore: update binding files and minor tweaks#279
Brooooooklyn merged 1 commit intomainfrom
10-30-chore_update_binding_files_and_minor_tweaks

Conversation

@Brooooooklyn
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings October 30, 2025 11:08
Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Brooooooklyn Brooooooklyn requested a review from fengmk2 October 30, 2025 11:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the @napi-rs/cli package from version 3.1.5 to 3.4.1 and adds necessary configuration changes to support this upgrade. The changes include updating dependency versions in the pnpm workspace catalog, excluding @napi-rs/* packages from minimum release age checks, implementing catalog key sorting, and updating the native binding loader with enhanced platform support.

  • Updated @napi-rs/cli from 3.1.5 to 3.4.1 with transitive dependency updates
  • Added @napi-rs/* to minimumReleaseAgeExclude for faster dependency updates
  • Enhanced native binding loader with Windows GNU and Linux loong64 architecture support

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
pnpm-workspace.yaml Updated @napi-rs/cli version to 3.4.1 and added @napi-rs/* to minimum release age exclusions
pnpm-lock.yaml Updated lockfile with new versions for @napi-rs/cli and all related inquirer/napi-rs packages
packages/tools/src/sync-remote-deps.ts Added alphabetical sorting for catalog keys
packages/cli/binding/index.js Enhanced native binding loader with Windows GNU support, loong64 architecture, and improved error handling
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/cli/binding/index.js
Comment thread packages/cli/binding/index.js
Comment thread packages/cli/binding/index.js
@Brooooooklyn Brooooooklyn merged commit 363aa90 into main Oct 30, 2025
24 of 25 checks passed
Copy link
Copy Markdown
Member Author

Merge activity

@Brooooooklyn Brooooooklyn deleted the 10-30-chore_update_binding_files_and_minor_tweaks branch October 30, 2025 11:49
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.

3 participants