fix: adapt elf workflow to remote rock build#75341
fix: adapt elf workflow to remote rock build#75341roryabraham merged 6 commits intoExpensify:mainfrom
Conversation
|
cc @roryabraham |
| - name: Download Rock artifact | ||
| run: | | ||
| echo "Downloading artifact from: $ARTIFACT_URL" | ||
| curl -L "$ARTIFACT_URL" -o artifact.zip |
There was a problem hiding this comment.
question: Would actions/download-artifact work as opposed to a manual curl?
There was a problem hiding this comment.
question: Where is the $ARTIFACT_URL var coming from?
There was a problem hiding this comment.
question: Where is the $ARTIFACT_URL var coming from?
$ARTIFACT_URL comes from https://github.com/Expensify/App/actions/runs/19438374744/job/55615206035#step:5:13465
question: Would actions/download-artifact work as opposed to a manual curl?
If I understand correctly, download-artifact works only when used within the same flow after using upload-artifact. Given that Rock does not upload the artifact using actions/upload-artifact, I believe it is impossible to download it this way.
|
@Kureev Validate GH actions is failing. Looks like a ShellCheck issue |
|
ignoring checklists for this GitHub Actions PR |
|
@roryabraham looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
|
🚀 Deployed to staging by https://github.com/roryabraham in version: 9.2.62-0 🚀
|
|
Hi @Kureev No QA for this, right? |
|
🚀 Deployed to production by https://github.com/marcaaron in version: 9.2.62-5 🚀
|
Explanation of Change
Adapt remote-build-android to remote rock build
Fixed Issues
$ #63871
PROPOSAL:
Tests
Offline tests
QA Steps
// TODO: These must be filled out, or the issue title must include "[No QA]."
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.npm run compress-svg)Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop