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

Re-add example CI #77

Merged
merged 3 commits into from
Sep 25, 2024
Merged

Re-add example CI #77

merged 3 commits into from
Sep 25, 2024

Conversation

luispadron
Copy link
Collaborator

@luispadron luispadron commented Sep 25, 2024

#75 removed CI for the example because they had gone stale and GitHub workers no longer support iOS 13 or macos-11. To not block other work (such as better support for SPM) I removed these tests. We should ideally continue to test the examples but requires more work.

This will require updates to support one of the macos & iOS platforms available to GitHub action workers: https://github.com/actions/runner-images/tree/main/images/macos We'll also need to update snaphot images.

@luispadron
Copy link
Collaborator Author

luispadron commented Sep 25, 2024

@dfed
Copy link
Collaborator

dfed commented Sep 25, 2024

Makes sense! I know @NickEntin has been excited about upgrading the snapshot images to latest as well. Certainly not the easiest lift.

@NickEntin NickEntin marked this pull request as ready for review September 25, 2024 07:33
@NickEntin NickEntin self-requested a review September 25, 2024 07:33
@NickEntin
Copy link
Collaborator

@luispadron Looks like I was removed as an admin on the repo - can you add me back or mark the Pod Lint and Xcode Build jobs as required? Also would be good to add @dfed to contributors so he can do approving code reviews. Thanks!

@luispadron luispadron merged commit 2bdbff8 into master Sep 25, 2024
3 checks passed
@luispadron
Copy link
Collaborator Author

Thanks for getting this updated! Added @dfed as maintainer

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.

3 participants