Skip to content
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

Adopt Xcode 16. Drop support for Xcode 15 #112

Merged
merged 2 commits into from
Sep 25, 2024
Merged

Adopt Xcode 16. Drop support for Xcode 15 #112

merged 2 commits into from
Sep 25, 2024

Conversation

dfed
Copy link
Owner

@dfed dfed commented Sep 23, 2024

This is a breaking change. I have no planned additional features for Xcode 15 / Swift 5, so it feels reasonable to cut a major version and drop support in new releases.

Copy link

codecov bot commented Sep 23, 2024

Codecov Report

Attention: Patch coverage is 95.83333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 99.52%. Comparing base (ec41b96) to head (1922796).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
Sources/SafeDICore/Models/TypeDescription.swift 84.61% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
- Coverage   99.67%   99.52%   -0.16%     
==========================================
  Files          33       33              
  Lines        2778     2713      -65     
==========================================
- Hits         2769     2700      -69     
- Misses          9       13       +4     
Files with missing lines Coverage Δ
Sources/SafeDICore/Models/Initializer.swift 100.00% <100.00%> (ø)
Sources/SafeDICore/Models/Property.swift 99.02% <100.00%> (-0.28%) ⬇️
Sources/SafeDICore/Models/TypeDescription.swift 98.61% <84.61%> (-0.58%) ⬇️

... and 1 file with indirect coverage changes

@dfed dfed force-pushed the dfed--xcode-16 branch 5 times, most recently from 69b92e2 to 8716e6b Compare September 23, 2024 03:06
Comment on lines +22 to +24
#if canImport(FoundationNetworking)
import FoundationNetworking
#endif
Copy link
Owner Author

Choose a reason for hiding this comment

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

networking is split out from Foundation on Linux, apparently

Comment on lines -78 to +81
chmod(downloadedURL.path(), mode_t(currentPermissions.uint16Value | S_IXUSR | S_IXGRP | S_IXOTH)) == 0
chmod(downloadedURL.path(), mode_t(currentPermissions.uint32Value) | S_IXUSR | S_IXGRP | S_IXOTH) == 0
Copy link
Owner Author

Choose a reason for hiding this comment

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

On Linux mode_t is a UInt32. On Apple's platforms it is a UInt16. So I need to wrap the mode_t cast on the currentPermissions value alone, and I figured I'd downscale the UInt32 value rather upscaling the UInt16 value. Doesn't really make a difference in the end though.

Comment on lines +31 to +34
sudo xcodebuild -runFirstLaunch
sudo xcrun simctl list
sudo xcodebuild -downloadPlatform visionOS
sudo xcodebuild -runFirstLaunch
Copy link
Owner Author

Choose a reason for hiding this comment

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

Per actions/runner-images#10559 what we had before was incomplete.

@dfed dfed marked this pull request as ready for review September 23, 2024 05:34
@dfed
Copy link
Owner Author

dfed commented Sep 25, 2024

Merging this, but happy to take review comments post-merge. Not planning on releasing this change until a few more things land.

@dfed dfed merged commit b50ff74 into main Sep 25, 2024
13 checks passed
@dfed dfed deleted the dfed--xcode-16 branch September 25, 2024 05:34
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.

1 participant