-
Notifications
You must be signed in to change notification settings - Fork 76
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
Upload project sources and submit archives to GCS #1524
Conversation
75072aa
to
57f3710
Compare
Size Change: +2.04 kB (0%) Total Size: 40.3 MB
|
75de07d
to
5dadd29
Compare
Codecov Report
@@ Coverage Diff @@
## main #1524 +/- ##
==========================================
+ Coverage 51.48% 51.49% +0.02%
==========================================
Files 453 453
Lines 15648 15656 +8
Branches 3076 3079 +3
==========================================
+ Hits 8055 8061 +6
- Misses 7579 7581 +2
Partials 14 14
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
resolvedArchiveSource: { type: SubmissionArchiveSourceType.Url, url }, | ||
source, |
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.
Couldn't this be sth like:
resolvedArchiveSource: { type: SubmissionArchiveSourceType.Url, url }, | |
source, | |
source: { type: SubmissionArchiveSourceType.Url, url }, |
resolvedArchiveSource: { type: SubmissionArchiveSourceType.GcsSubmitArchive, bucketKey }, | ||
source, |
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.
And:
resolvedArchiveSource: { type: SubmissionArchiveSourceType.GcsSubmitArchive, bucketKey }, | |
source, | |
source: { type: SubmissionArchiveSourceType.GcsSubmitArchive, bucketKey }, |
If I understand correctly, this needs another Archive...
interface.
I don't think we should have two almost identical fields in the return object.
const fileStat = await fs.stat(file); | ||
const fileSize = fileStat.size; |
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.
const fileStat = await fs.stat(file); | |
const fileSize = fileStat.size; | |
const { size: fileSize } = await fs.stat(file); |
@@ -8,7 +8,7 @@ | |||
"build:tarballs:linux": "yarn build-for-size-check", | |||
"build-for-size-check": "yarn build && yarn workspace eas-cli pretarball-ci && CLI_SIZE_CHECK=1 yarn workspace eas-cli oclif pack:tarballs --no-xz --targets linux-x64", | |||
"typecheck": "lerna run typecheck", | |||
"start": "lerna run watch --parallel", | |||
"start": "lerna run watch --stream --parallel", |
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.
new lerna version does not show output without stream option
fe82c63
to
5bff6e7
Compare
/changelog-entry chore Upload project sources for EAS Build and archives for EAS Submit to GCS. |
Checklist
/changelog-entry [breaking-change|new-feature|bug-fix|chore] [message]
and CHANGELOG.md will be updated automatically.Why
Upload project sourced for build and ipa/aab for submits to gcs
How
Test Plan
Run build, build with auto submit, submit with path option, submit from existing build