-
Notifications
You must be signed in to change notification settings - Fork 83
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
[eas-cli] rework expected bundle format for eas publish #196
Conversation
Size Change: -2.92 kB (0%) Total Size: 35.8 MB
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a few comments below, nothing major – the PR looks pretty good!
launchAsset: { | ||
type: 'bundle', | ||
contentType: 'application/javascript', | ||
path: path.resolve(distRoot, metadata.fileMetadata[platform].bundle), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this might result in Cannot read property 'bundle' of undefined
, if the bundle doesn't include all platforms? I wonder if we should check that the key exist?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point, added validation with @hapi/joi
Co-authored-by: Ville Immonen <ville.immonen@iki.fi>
8576383
to
3ab91c2
Compare
Why
Previously we were relying on
expo export
to build the bundles published byeas publish
.expo export
turns out to do a lot of unrelated stuff with the assumption the developer is going to use the output for privately hosted static endpoints. The core elements ofexpo export
were factored out and put into a new commandexpo bundle
expo/expo-cli#3074.expo bundle
outputs the bundle in a new format. This PR updateseas publish
to expect this new format.How
Rewrote
collectAssets
.Test Plan
Confirmed the
collectAssets
output test still passed after updating it to the new formate.Made a test publish after building the bundle in the new format.