fix for Xcode 27 - #74
Merged
Merged
Conversation
gestrich
approved these changes
Sep 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Enable building with Xcode 27
Method
The variables in one file were defined after they were used.
Xcode 27 is more strict about this syntax.
I simply moved some lines in this one file: LoopCaregiver/LoopCaregiver/Views/Actions/OverrideInsulinNeedsView.swift
Test
The
devbranch failed to build using Xcode 27.The
fix/xcode27branch built successfully and installed on an SE 3rd gen phone running iOS 27.The remote commands were successful
Not sure if we want to update any of the packages. Happy to do that and test if requested.