Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REF-2710]Recommend Running with REFLEX_USE_NPM=1 if npm run fails after installing packages with bun. #3399

Merged
merged 1 commit into from
May 30, 2024

Conversation

ElijahAhianyo
Copy link
Collaborator

For errors that happen when running npm run if bun was used as a package manager(example below).

Debug: Running command: ['C:\\Users\\serrap\\AppData\\Local\\reflex\\fnm\\node-versions\\v18.17.0\\installation\\npm', 'run', 'dev']
Debug: Starting frontend
Debug: 
Debug: > dev
Debug: > next dev
Debug:
Debug: error: bin executable does not exist on disk
Debug:
Debug: Bun failed to remap this bin to its proper location within node_modules.
Debug: This is an indication of a corrupted node_modules directory.
Debug:
Debug: Please run 'bun install --force' in the project root and try
Debug: it again. If this message persists, please open an issue:
Debug: https://github.com/oven-sh/bun/issues
Debug:
Starting frontend failed with exit code 255
> dev
> next dev
error: bin executable does not exist on disk
Bun failed to remap this bin to its proper location within node_modules.
This is an indication of a corrupted node_modules directory.
Please run 'bun install --force' in the project root and try
it again. If this message persists, please open an issue:
https://github.com/oven-sh/bun/issues
Run with --loglevel debug  for the full log.

We should detect these and recommend users run reflex with REFLEX_USE_NPM to use npm only

Copy link

linear bot commented May 29, 2024

@masenf masenf merged commit 1a833d6 into main May 30, 2024
47 checks passed
benedikt-bartscher pushed a commit to benedikt-bartscher/reflex that referenced this pull request Jun 3, 2024
@masenf masenf deleted the elijah/bun-reflex-use-npm-warning branch December 12, 2024 07:19
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