Repository navigation
Fix compile errors for Swift 6 language mode - #230
Conversation
natecook1000
left a comment
natecook1000
left a comment
There was a problem hiding this comment.
Thanks so much for this, @b1ackturtle — just one set of changes and we can merge this.
| elements: S, | ||
| expectedValue: Int, | ||
| file: StaticString = #file, line: UInt = #line | ||
| file: StaticString = #filePath, line: UInt = #line |
There was a problem hiding this comment.
Could we resolve these warnings in this way instead? The #filePath attribute was introduced in Swift 5.8, but we're going to support back to Swift 5.7 in the next release of this package.
| file: StaticString = #filePath, line: UInt = #line | |
| file: StaticString = (#file), line: UInt = #line |
There was a problem hiding this comment.
Thank you for your review. I have updated my code based on your suggestion.
b1cf3c3 to
135802f
Compare
|
@swift-ci please test |
|
|
||
| final class GroupedTests: XCTestCase { | ||
| private class SampleError: Error {} | ||
| private final class SampleError: Error {} |
There was a problem hiding this comment.
Can you please explain, why do we need this final keyword here?
There was a problem hiding this comment.
It was added to resolve the following error:
Non-final class 'SampleError' cannot conform to 'Sendable'; use @unchecked Sendable
There was a problem hiding this comment.
Thanks for explanation. Can you please also update you branch with base branch?
135802f to
c378d3c
Compare
|
@swift-ci Please test |
|
@swift-ci Please test |
This PR fixes code that caused compile errors when enabling the Swift 6 language mode.
Checklist