Skip to content

Update SPM dependencies #550

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update SPM dependencies #550

wants to merge 2 commits into from

Conversation

ehab-al-cko
Copy link
Contributor

@ehab-al-cko ehab-al-cko commented May 27, 2025 •
edited
Loading

Issue

PIMOB-3566

Proposed changes

  • Upgrade PhoneNumberKit to "version" : "4.0.2"
  • Update Risk SDK min version "3.0.2"
  • Update min iOS version to be iOS 13.0
  • Update GHA to macos15, xcode 16 and iphone 16 pro
  • Support secure to use uses: actions/checkout@85e6279instead of@V3 or @v4`
  • added this solution because it's failing on MainActor

@ehab-al-cko ehab-al-cko changed the base branch from develop to main May 27, 2025 14:53
@ehab-al-cko ehab-al-cko self-assigned this May 27, 2025
@ehab-al-cko ehab-al-cko force-pushed the update/spm_dependacies branch 5 times, most recently from 1b23a27 to 8a2cc53 Compare May 28, 2025 13:57
@ehab-al-cko ehab-al-cko had a problem deploying to AppStoreValidation May 28, 2025 14:10 — with GitHub Actions Failure
@ehab-al-cko ehab-al-cko requested a review from a team May 28, 2025 14:22
@ehab-al-cko ehab-al-cko force-pushed the update/spm_dependacies branch from 8a2cc53 to b5a34e7 Compare May 28, 2025 15:08
okhan-okbay-cko
okhan-okbay-cko reviewed May 28, 2025
View reviewed changes
@ehab-al-cko ehab-al-cko force-pushed the update/spm_dependacies branch from b5a34e7 to b527977 Compare May 29, 2025 12:02
@ehab-al-cko ehab-al-cko force-pushed the update/spm_dependacies branch from b527977 to 50fc933 Compare May 29, 2025 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@okhan-okbay-cko okhan-okbay-cko okhan-okbay-cko left review comments

@ali-farhadi-cko ali-farhadi-cko ali-farhadi-cko approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

@ehab-al-cko ehab-al-cko

Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ehab-al-cko @okhan-okbay-cko @ali-farhadi-cko