Skip to content

Revert "Back to Bun (it now supports arm builds)"#938

Merged
martpie merged 1 commit intomasterfrom
npm
Mar 7, 2026
Merged

Revert "Back to Bun (it now supports arm builds)"#938
martpie merged 1 commit intomasterfrom
npm

Conversation

@martpie
Copy link
Owner

@martpie martpie commented Mar 7, 2026

This reverts commit 0174b88.

Alternative to #937, see motivation there

Copilot AI review requested due to automatic review settings March 7, 2026 19:15
Copy link
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 reverts the project’s “back to Bun” tooling change by moving the dev/build workflow back to Node.js + npm, updating related docs/config, and removing Bun-specific artifacts.

Changes:

  • Switch Tauri build hooks, Git hooks, docs, and CI workflows from bun commands to npm commands.
  • Remove Bun-specific dependency/types and delete the bun.lock lockfile.
  • Add Node/npm version expectations via devEngines in package.json.

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/translations/languages.ts Updates contributor instructions to use npm run gen:translations.
src-tauri/tauri.conf.json Updates frontend build commands to npm run ... (and removes JSON schema line).
src-tauri/src/plugins/file_associations.rs Updates Windows dev test command example to use npm.
package.json Switches pre-commit hook command to npm, removes @types/bun, adds devEngines.
bun.lock Removes Bun lockfile.
README.md Updates setup/dev/build docs from Bun to Node/npm.
AGENTS.md Updates documented dev commands from Bun to npm.
.npmrc Updates npm config (adds audit=false).
.github/workflows/build.yml Switches CI from setup-bun + Bun commands to setup-node + npm commands.

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

@martpie martpie merged commit 53d99ef into master Mar 7, 2026
6 checks passed
@martpie martpie deleted the npm branch March 7, 2026 21:41
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.

2 participants