Closed bobgilmore closed 5 years ago
5 Warnings | |
---|---|
:warning: | ResultTests.swift#L15 - SwiftWisdomTests/CommonTypes/ResultTests.swift#L15: ‘IPResult’ is deprecated: use Swift.Result instead
private final let successfulResult = IPResult<Int>.success(1)
|
:warning: | ResultTests.swift#L16 - SwiftWisdomTests/CommonTypes/ResultTests.swift#L16: ‘IPResult’ is deprecated: use Swift.Result instead
private final let failureResult = IPResult<Any>.failure(ResultError.error)
|
:warning: | UnsignedInteger+ExtensionsTests.swift#L16 - SwiftWisdomTests/StandardLibrary/Numbers/UnsignedInteger+ExtensionsTests.swift#L16: ‘random(inRange:)’ is deprecated: Use Int.random(in:) instead
XCTAssert(positives.contains(random(inRange: positives)))
|
:warning: | UnsignedInteger+ExtensionsTests.swift#L18 - SwiftWisdomTests/StandardLibrary/Numbers/UnsignedInteger+ExtensionsTests.swift#L18: ‘random(inRange:)’ is deprecated: Use Int.random(in:) instead
XCTAssert(negatives.contains(random(inRange: negatives)))
|
:warning: | UnsignedInteger+ExtensionsTests.swift#L20 - SwiftWisdomTests/StandardLibrary/Numbers/UnsignedInteger+ExtensionsTests.swift#L20: ‘random(inRange:)’ is deprecated: Use Int.random(in:) instead
XCTAssert(mixed.contains(random(inRange: mixed)))
|
1 Message | |
---|---|
:book: | Executed 174 tests, with 0 failures (0 unexpected) in 0.434 (0.490) seconds |
Generated by :no_entry_sign: Danger
I'm going to squash this down to three commits, THEN push...
.swiftlint.yml
This updates our two-year-old .swiftlint.yml to the latest version in https://github.com/IntrepidPursuits/swift-style-guide/blob/master/.swiftlint.yml , and cleans up the resulting warnings and errors.
I've created this PR with each class of warning or error handled in a separate commit, for ease of review. When I merge, I plan to squash all of the code changes down into one commit.