-
Notifications
You must be signed in to change notification settings - Fork 12
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
Dev v5.0.0 to staging #53
Conversation
update for major version 5.0
Log tag is used by other extensions for migration logic logging.
Fix tests and build failure
…s into dev-v5.0.0
Update Xcode version in GH Actions
Point to staging for testing 5.x
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a question on xcode version used for our ci builds
|
||
steps: | ||
- uses: actions/checkout@v2 | ||
|
||
- uses: maxim-lobanov/setup-xcode@v1 | ||
with: | ||
xcode-version: 14.1 | ||
xcode-version: 15.0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
15.1.0?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I kept it on 15.0.1 because the slow performance didn't seem to affect this extensions tests passing. I'm open to swapping to 15.1 to match other extensions though? I mostly thought it would be nice to keep on parity with the release Xcode version if possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I kept it on 15.0.1 because the slow performance didn't seem to affect this extensions tests passing. I'm open to swapping to 15.1 to match other extensions though? I mostly thought it would be nice to keep on parity with the release Xcode version if possible.
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: