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

CI script to setup/test aarch64-apple-ios-sim #340

Merged
merged 2 commits into from
Feb 23, 2024

Conversation

justinwsmith
Copy link
Contributor

@justinwsmith justinwsmith commented Feb 13, 2024

Description of changes:

  • Created a script that sets up environment to test aarch64-apple-ios-sim
    • Locates appropriate iOS simulator runtime image
    • Mounts the image
    • Executes the tests.

Testing:

  • Verified it works in Github Workflow. Script also works on my M1 Mac

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@justinwsmith justinwsmith requested a review from a team as a code owner February 13, 2024 20:26
@codecov-commenter
Copy link

codecov-commenter commented Feb 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.82%. Comparing base (5d13428) to head (c7d090b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #340   +/-   ##
=======================================
  Coverage   95.82%   95.82%           
=======================================
  Files          60       60           
  Lines        7865     7865           
=======================================
  Hits         7537     7537           
  Misses        328      328           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@justsmth justsmth force-pushed the ios-sim-DYLD_ROOT_PATH branch 2 times, most recently from c93df77 to de02e5c Compare February 13, 2024 21:22
@justsmth justsmth force-pushed the ios-sim-DYLD_ROOT_PATH branch 2 times, most recently from 5f6da63 to c99a5b3 Compare February 13, 2024 21:47
@justsmth justsmth marked this pull request as draft February 13, 2024 21:50
@justsmth justsmth force-pushed the ios-sim-DYLD_ROOT_PATH branch 2 times, most recently from c1ed9c4 to 843dd69 Compare February 20, 2024 17:11
@justsmth justsmth changed the title Update DYLD_ROOT_PATH Build, no tests, for aarch64-apple-ios-sim until we find alternative Feb 20, 2024
@justsmth justsmth marked this pull request as ready for review February 20, 2024 17:14
samuel40791765
samuel40791765 previously approved these changes Feb 21, 2024
@justsmth justsmth marked this pull request as draft February 22, 2024 17:13
@justinwsmith justinwsmith force-pushed the ios-sim-DYLD_ROOT_PATH branch 5 times, most recently from 85b9358 to 9834b61 Compare February 22, 2024 22:50
@justinwsmith justinwsmith changed the title Build, no tests, for aarch64-apple-ios-sim until we find alternative Script to setup/test aarch64-apple-ios-sim Feb 22, 2024
@justinwsmith justinwsmith changed the title Script to setup/test aarch64-apple-ios-sim CI script to setup/test aarch64-apple-ios-sim Feb 22, 2024
@justinwsmith justinwsmith marked this pull request as ready for review February 22, 2024 22:58
fi
done

if [[ -z ${IMAGE_PATH} ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
if [[ -z ${IMAGE_PATH} ]]; then
if [[ -z "${IMAGE_PATH}" ]]; then

consistency nit

@justsmth justsmth merged commit 65c9d11 into aws:main Feb 23, 2024
149 of 153 checks passed
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.

5 participants