-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[No QA] Callable buildIOS workflow #50123
Conversation
npm has a |
d8998c4
to
5ebe423
Compare
This comment has been minimized.
This comment has been minimized.
5ebe423
to
6bdeb2c
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪 |
.github/workflows/buildIOS.yml
Outdated
- name: Upload iOS sourcemaps | ||
uses: actions/upload-artifact@v4 | ||
with: | ||
name: ios-sourcemaps-artifact |
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.
Should it be ios-artifact-sourcemaps
?
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.
Also do we do anything with this?
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.
I kept it as ios-sourcemaps-artifact
intentionally, because that's what it was called before. I'm not sure but I think we might download this artifact as part of some debug/development tools (which is why I didn't want to rename it unnecessarily). I can look into it
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.
In the only other reference in this repo, it's just uploaded to the GitHub Release. I think it might be downloaded from there manually to debug issues reported in #newdot-quality though
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.
Ah - The only reason I said is because I'm not sure we will download it pattern: ios-artifact-*
? But if it's downloaded elsewhere already, that sounds good
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.
ended up renaming this to ios-artifact-sourcemaps
for consistency. It will still get uploaded to the GitHub Release
noticed a 🐛, fixing it |
fixed! |
lol, found another 😂 |
Fixed! |
Reviewer Checklist
Screenshots/VideosAndroid: NativeN/A - No user code Android: mWeb ChromeN/A - No user code iOS: NativeN/A - No user code iOS: mWeb SafariN/A - No user code MacOS: Chrome / SafariN/A - No user code MacOS: DesktopN/A - No user code |
Not going to wait for performance tests, going to merge this one to get it tested. |
@roryabraham looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
[No QA] Callable buildIOS workflow (cherry picked from commit 6606f1d) (CP triggered by roryabraham)
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪
|
🚀 Cherry-picked to staging by https://github.com/roryabraham in version: 9.0.44-8 🚀
|
🚀 Cherry-picked to staging by https://github.com/roryabraham in version: 9.0.44-10 🚀
|
🚀 Deployed to production by https://github.com/jasperhuangg in version: 9.0.44-12 🚀
|
Details
Creates a callable iOS build workflow.
Fixed Issues
$ n/a - related to CP to prod
Tests
We'll have to live-test the main deploy but we can test this branch with AdHoc builds to make sure that works.
Offline tests
n/a
QA Steps
none
PR Author Checklist
### Fixed Issues
section aboveTests
sectionOffline steps
sectionQA steps
sectiontoggleReport
and notonIconClick
)myBool && <MyComponent />
.src/languages/*
files and using the translation methodSTYLE.md
) were followedAvatar
, I verified the components usingAvatar
are working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)
)Avatar
is modified, I verified thatAvatar
is working as expected in all cases)Design
label and/or tagged@Expensify/design
so the design team can review the changes.ScrollView
component to make it scrollable when more elements are added to the page.main
branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTest
steps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop