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

Bump eas-cli-local-build-plugin to latest version to support new bun lock file #2817

Merged

Conversation

shiroyasha9
Copy link
Contributor

@shiroyasha9 shiroyasha9 commented Jan 14, 2025

Why

As mentioned in my original issue (#2782 (comment)), Bun introduced a new text based bun.lock file with their Bun v1.1.39+.

This new lock file was supported by EAS in the cloud builds as seen in PRs #2801 abd expo/eas-build#482, and indeed upon verification works perfectly fine in cloud builds (https://expo.dev/accounts/mubin-memorang/projects/new-bun-lock/builds/a8f5d20f-20b8-4b20-9430-cbf48ba88103)

This however was not working with the --local flag.

Upon inspection, the eas-cli-local-build-plugin is not updated to the latest version that reflects this new lockfile related changes.

After bumping the package

Gets recognized as a valid bun project

image

Builds gets successful

image

Copy link

Subscribed to pull request

File Patterns Mentions
**/* @szdziedzic, @khamilowicz, @sjchmiela, @radoslawkrzemien

Generated by CodeMention

Copy link
Member

@szdziedzic szdziedzic left a comment

Choose a reason for hiding this comment

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

thanks!

@szdziedzic szdziedzic merged commit 1c3edbe into expo:main Jan 15, 2025
6 of 7 checks passed
@evelant
Copy link

evelant commented Jan 23, 2025

Did this not make it into 14.5.0 or is there possibly another bug? eas-cli 14.5.0 isn't picking up bun.lock for local builds. It tries to use yarn "We detected that 'apps/myapp' is a yarn workspace" resulting in a failure.

@shiroyasha9
Copy link
Contributor Author

@evelant it's working for me, is it possible to create a quick repro? I can take a look at it then

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