Skip to content

Re-add example CI#77

Merged
luispadron merged 3 commits into
masterfrom
luis/add-back-example-ci
Sep 25, 2024
Merged

Re-add example CI#77
luispadron merged 3 commits into
masterfrom
luis/add-back-example-ci

Conversation

@luispadron

@luispadron luispadron commented Sep 25, 2024

Copy link
Copy Markdown
Collaborator

#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

luispadron commented Sep 25, 2024

Copy link
Copy Markdown
Collaborator Author

@dfed

dfed commented Sep 25, 2024

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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
@luispadron

Copy link
Copy Markdown
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