Skip to content

fix(rust-napi-ci): default install command to bun --no-verify#31

Merged
jan-kubica merged 1 commit into
mainfrom
fix/rust-napi-ci-no-verify
May 15, 2026
Merged

fix(rust-napi-ci): default install command to bun --no-verify#31
jan-kubica merged 1 commit into
mainfrom
fix/rust-napi-ci-no-verify

Conversation

@jan-kubica
Copy link
Copy Markdown
Contributor

Summary

Adds an install-command input to the reusable rust-napi-ci.yml (default bun install --frozen-lockfile --no-verify) and routes every bun install step through it. The default works around a bun integrity-check failure on napi-rs platform sub-packages that currently breaks every consumer's Lint, Pack, Runtime, and Build jobs (e.g. stella/aho-corasick PR #69, stella/fuzzy-search PR #48, stella/regex-set PR #78). Mirrors the existing pattern in provenance-update.yml.

Test plan

  • After merge, bump SHA in consumer ci.yml files (aho-corasick, fuzzy-search, regex-set) and confirm Lint/Pack/Runtime/Build jobs go green without per-consumer overrides.

Adds an `install-command` input (default
`bun install --frozen-lockfile --no-verify`) and threads it through
every `bun install` invocation. The default works around a bun
integrity-check failure on napi-rs platform sub-packages that
otherwise breaks every napi-ci consumer's Lint/Pack/Runtime/Build
jobs. Callers can override the input if they need different behaviour.
@jan-kubica jan-kubica requested a review from nnad3N as a code owner May 15, 2026 08:12
@gemini-code-assist
Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@jan-kubica jan-kubica merged commit e78d235 into main May 15, 2026
1 check passed
@jan-kubica jan-kubica deleted the fix/rust-napi-ci-no-verify branch May 15, 2026 08:30
@github-actions github-actions Bot locked and limited conversation to collaborators May 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant