-
-
Notifications
You must be signed in to change notification settings - Fork 49
Build with Swift 6.1 #3750
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
Build with Swift 6.1 #3750
Conversation
# Conflicts: # .devcontainer/Dockerfile # .github/workflows/ci.yml # .github/workflows/query-performance.yml # Dockerfile # LOCAL_DEVELOPMENT_SETUP.md # Makefile
import Testing | ||
|
||
|
||
@Suite( | ||
.dependency(\.date.now, .t0), | ||
.dependency(\.metricsSystem, .mock) | ||
.dependency(\.metricsSystem, .mock), | ||
.snapshots(record: .failed) |
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.
This is a great quality of life change in SnapshotTesting: The if a snapshot test fails, it will automatically record a new snapshot.
That means that in order to update snapshots, we don't need to edit tests anymore. We just need to check if there are snapshot differences and commit or revert them as needed.
}, | ||
"traits" : [ | ||
|
||
] |
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.
Swift 6.1 package dump format changes.
Revert "Merge pull request #3750 from SwiftPackageIndex/swift-6.1"
This reverts commit bd6b110.
This reverts commit bd6b110.
This just switches the building and running of spi-server to 6.1.
I'm pulling this forward, because it's needed for parallel testing (and also the Xcode 16.3 update).