Skip to content

chore(ci): Update macOS and Xcode - #2633

Merged
jcesarmobile merged 1 commit into
nextfrom
RMET-4714
Nov 9, 2026
Merged

jcesarmobile merged 1 commit into
nextfrom
RMET-4714

Conversation

@jcesarmobile

Copy link
Copy Markdown
Member

No description provided.

Comment thread .github/workflows/ci.yml
Comment on lines 114 to 115
- run: xcrun simctl list > /dev/null
- run: xcodebuild -downloadPlatform iOS

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I didn't check but perhaps in this new runner these couple of lines aren't needed anymore?

Doesn't need to be addressed in this PR but just thought of it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

might not be needed now, but since we stick to 27.0 for the whole year, they will most likely remove olde simulators when they add new ones

Comment thread .github/workflows/ci.yml

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Doesn't seem like CI ran with updated Xcode version because there are no plugin changes, just checking if we've done that or not, otherwise can we be sure that the CI is working?

I suppose we can find that out whenever we make plugin changes in the future, but doesn't hurt to check now?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

check how? make some change in all plugins and then revert it?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I've done that in the past myself yeah. Since we're squashing commits it doesn't really reflect in the end result.

But if you think it's not necessary then it can be merged as-is. I'll approve the PR.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've run verify:ios locally with Xcode 27.0 and it built correcly, so I think we are fine.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Fair enough!

@jcesarmobile
jcesarmobile merged commit df4267f into next Nov 9, 2026
9 checks passed
@jcesarmobile
jcesarmobile deleted the RMET-4714 branch November 9, 2026 11:00
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.

2 participants