chore: bump Swift to 6.2 and SwiftFormat to 0.60.1#152
Merged
Conversation
Swift 6.2 is the new minimum after the tools version bump. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Package.resolved is gitignored, so CI was resolving the latest SwiftFormat matching `from: "0.54.1"` (0.60.1) while local kept the previously pinned 0.58.1. That newer version introduced additional rules (swiftTestingTestCaseNames, simplifyGenericConstraints, redundantThrows, redundantSendable, stricter docComments / indent / trailingSpace handling) which caused CI lint to fail. Raise the minimum to 0.60.1 so local and CI converge, and apply the resulting formatting fixes across Sources/ and Tests/. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Pin SwiftFormat (and the other dev dependencies) at a known version so CI and local builds agree. Without this, CI was resolving the latest SwiftFormat matching `from:` on every run, which silently introduced new lint rules the moment a new release shipped. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
bok-
changed the title
|
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
1 participant
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.
Summary
Package.swift,Examples/Package.swift,.swift-version) and reformats sources to match the Swift 6.2 formatter outputSwiftFormattofrom: "0.60.1"and applies the formatting fixes that new version requires (new rules:swiftTestingTestCaseNames,redundantThrows,redundantSendable,simplifyGenericConstraints, plus tighteneddocComments/indent/trailingSpacehandling)Package.resolvedinto source control so CI and local resolve identical dependency versions — previously it was gitignored, which meant CI was picking up new SwiftFormat releases the moment they shippedWhy the SwiftFormat + Package.resolved change
CI lint started failing on this branch with ~200 errors that didn't reproduce locally. Root cause:
/Package.resolvedwas in.gitignore, so local stayed pinned at SwiftFormat 0.58.1 while CI resolved 0.60.1 fresh on every run and enforced its new rules. Pinning viaPackage.resolvedprevents this class of drift for all dev dependencies going forward.Test plan
swift testpasses locally (174/174) under Xcode 26.3 (Swift 6.2)make lintpasses locally under Xcode 26.3