Update min deployment target of Xcode schemes - #1591
Conversation
|
Important Review skippedReview was skipped due to path filters β Files ignored due to path filters (43)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including βοΈ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. π βΉοΈ Recent review infoβοΈ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: β Files ignored due to path filters (47)
π Files selected for processing (10)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. π WalkthroughWalkthroughThe change adds environment-controlled liquid glass rendering, effect-free snapshot helpers and test migrations, and iOS 15 deployment targets for affected framework and test targets. ChangesLiquid Glass Controls and Snapshot Updates
iOS 15 Deployment Target Alignment
Priority: β¬οΈ Low β Defer this change because it updates SwiftUI deployment settings, liquid-glass handling, and snapshot rendering without changing public APIs or described runtime features. Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: βͺ Minimal Β· up to No actionable merge-blocking risk remains. Sequence Diagram(s)sequenceDiagram
participant SnapshotTest
participant StreamChatTestCase
participant SwiftUIEnvironment
participant LiquidGlassModifiers
SnapshotTest->>StreamChatTestCase: assertSnapshotWithoutVisualEffects(view)
StreamChatTestCase->>SwiftUIEnvironment: disable liquid glass effects
SwiftUIEnvironment->>LiquidGlassModifiers: provide disabled setting
LiquidGlassModifiers->>StreamChatTestCase: render without visual effects
StreamChatTestCase->>SnapshotTest: compare rendered image
Suggested reviewers: π₯ Pre-merge checks | β 4 | β 1β Failed checks (1 warning)
β Passed checks (4 passed)
β¨ Finishing Touches π‘ 1π Generate docstrings π‘
π§ͺ Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Generated by π« Danger |
SDK Size
|
StreamChatSwiftUI XCSize
Show 131 more objects
|
Public Interfaceπ No changes affecting the public interface. |
π Issue Links
Resolves https://linear.app/stream/issue/IOS-2010/build-demo-app-on-xcode-27.
π― Goal
Make the demo app build on Xcode 27.
π Summary
Provide bullet points with the most important changes in the codebase.
π Implementation
Provide a detailed description of the implementation and explain your decisions if you find them relevant.
π¨ Showcase
Add relevant screenshots and/or videos/gifs to easily see what this PR changes, if applicable.
π§ͺ Manual Testing Notes
Explain how this change can be tested manually, if applicable.
βοΈ Contributor Checklist
docs-contentrepoSummary by CodeRabbit
New Features
Compatibility
Tests