-
Notifications
You must be signed in to change notification settings - Fork 119
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
Switch to Xcode 15 (with the iOS 16.4 Simulator for tests still) #2113
Conversation
Generated by 🚫 Danger Swift against f78d22e |
c548176
to
8b6e47c
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2113 +/- ##
===========================================
- Coverage 71.92% 71.91% -0.01%
===========================================
Files 511 511
Lines 34722 34722
Branches 16673 16673
===========================================
- Hits 24973 24972 -1
Misses 9137 9137
- Partials 612 613 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Use iOS 16.4 simulators from Xcode 14 as iOS 17.0.1 simulators are **super** slow.
Snapshot fix.
8b6e47c
to
9028443
Compare
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.
🕺
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.
LGTM 🎉
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
LGTM!
Follow up from #2007, this switches the project to use Xcode 15, but with the iOS 16.4 simulator linked from Xcode 14 on CI to work around the performance issues.
Updated Preview snapshots are mainly about the default change of continuous corners on rounded rectangles and capsules. There are some rendering errors but those don't appear in real life.
UI tests and integration tests both work locally too and the precision is lower on UI tests so it doesn't pick up the corner shape change.