Skip to content

Commit e51fb00

Browse files
authored
Remove type alias for Xcode 12.5 compatibility (#371)
* Remove type alias * Fix a few warnings
1 parent 636f3fa commit e51fb00

File tree

4 files changed

+12
-16
lines changed

4 files changed

+12
-16
lines changed

Examples/CaseStudies/SwiftUICaseStudies/00-Core.swift

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,6 @@ struct RootEnvironment {
6969
var fetchNumber: () -> Effect<Int, Never>
7070
var mainQueue: AnySchedulerOf<DispatchQueue>
7171
var numberFact: (Int) -> Effect<String, NumbersApiError>
72-
var trivia: (Int) -> Effect<String, TriviaApiError>
7372
var userDidTakeScreenshot: Effect<Void, Never>
7473
var uuid: () -> UUID
7574
var webSocket: WebSocketClient
@@ -81,7 +80,6 @@ struct RootEnvironment {
8180
fetchNumber: liveFetchNumber,
8281
mainQueue: DispatchQueue.main.eraseToAnyScheduler(),
8382
numberFact: liveNumberFact(for:),
84-
trivia: liveTrivia(for:),
8583
userDidTakeScreenshot: liveUserDidTakeScreenshot,
8684
uuid: UUID.init,
8785
webSocket: .live
@@ -145,7 +143,7 @@ let rootReducer = Reducer<RootState, RootAction, RootEnvironment>.combine(
145143
.pullback(
146144
state: \.effectsCancellation,
147145
action: /RootAction.effectsCancellation,
148-
environment: { .init(mainQueue: $0.mainQueue, trivia: $0.trivia) }
146+
environment: { .init(mainQueue: $0.mainQueue, numberFact: $0.numberFact) }
149147
),
150148
episodesReducer
151149
.pullback(
@@ -271,7 +269,7 @@ func liveNumberFact(for n: Int) -> Effect<String, NumbersApiError> {
271269
Just("\(n) is a good number Brent")
272270
.delay(for: 1, scheduler: DispatchQueue.main)
273271
}
274-
.mapError { _ in NumbersApiError() }
272+
.setFailureType(to: NumbersApiError.self)
275273
.eraseToEffect()
276274
}
277275

@@ -287,7 +285,7 @@ func liveTrivia(for n: Int) -> Effect<String, TriviaApiError> {
287285
Just("\(n) is a good number Brent")
288286
.delay(for: 1, scheduler: DispatchQueue.main)
289287
}
290-
.mapError { _ in TriviaApiError() }
288+
.setFailureType(to: TriviaApiError.self)
291289
.eraseToEffect()
292290
}
293291

Examples/CaseStudies/SwiftUICaseStudies/02-Effects-Cancellation.swift

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,14 +25,14 @@ enum EffectsCancellationAction: Equatable {
2525
case cancelButtonTapped
2626
case stepperChanged(Int)
2727
case triviaButtonTapped
28-
case triviaResponse(Result<String, TriviaApiError>)
28+
case triviaResponse(Result<String, NumbersApiError>)
2929
}
3030

3131
struct TriviaApiError: Error, Equatable {}
3232

3333
struct EffectsCancellationEnvironment {
3434
var mainQueue: AnySchedulerOf<DispatchQueue>
35-
var trivia: (Int) -> Effect<String, TriviaApiError>
35+
var numberFact: (Int) -> Effect<String, NumbersApiError>
3636
}
3737

3838
// MARK: - Business logic
@@ -58,7 +58,7 @@ let effectsCancellationReducer = Reducer<
5858
state.currentTrivia = nil
5959
state.isTriviaRequestInFlight = true
6060

61-
return environment.trivia(state.count)
61+
return environment.numberFact(state.count)
6262
.receive(on: environment.mainQueue)
6363
.catchToEffect()
6464
.map(EffectsCancellationAction.triviaResponse)
@@ -128,7 +128,7 @@ struct EffectsCancellation_Previews: PreviewProvider {
128128
reducer: effectsCancellationReducer,
129129
environment: EffectsCancellationEnvironment(
130130
mainQueue: DispatchQueue.main.eraseToAnyScheduler(),
131-
trivia: liveTrivia(for:)
131+
numberFact: liveNumberFact(for:)
132132
)
133133
)
134134
)

Examples/CaseStudies/SwiftUICaseStudiesTests/02-Effects-CancellationTests.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ class EffectsCancellationTests: XCTestCase {
1313
reducer: effectsCancellationReducer,
1414
environment: .init(
1515
mainQueue: self.scheduler.eraseToAnyScheduler(),
16-
trivia: { n in Effect(value: "\(n) is a good number Brent") }
16+
numberFact: { n in Effect(value: "\(n) is a good number Brent") }
1717
)
1818
)
1919

@@ -43,7 +43,7 @@ class EffectsCancellationTests: XCTestCase {
4343
reducer: effectsCancellationReducer,
4444
environment: .init(
4545
mainQueue: self.scheduler.eraseToAnyScheduler(),
46-
trivia: { _ in Fail(error: TriviaApiError()).eraseToEffect() }
46+
numberFact: { _ in Fail(error: NumbersApiError()).eraseToEffect() }
4747
)
4848
)
4949

@@ -54,7 +54,7 @@ class EffectsCancellationTests: XCTestCase {
5454
.do {
5555
self.scheduler.advance()
5656
},
57-
.receive(.triviaResponse(.failure(TriviaApiError()))) {
57+
.receive(.triviaResponse(.failure(NumbersApiError()))) {
5858
$0.isTriviaRequestInFlight = false
5959
}
6060
)
@@ -72,7 +72,7 @@ class EffectsCancellationTests: XCTestCase {
7272
reducer: effectsCancellationReducer,
7373
environment: .init(
7474
mainQueue: self.scheduler.eraseToAnyScheduler(),
75-
trivia: { n in Effect(value: "\(n) is a good number Brent") }
75+
numberFact: { n in Effect(value: "\(n) is a good number Brent") }
7676
)
7777
)
7878

@@ -95,7 +95,7 @@ class EffectsCancellationTests: XCTestCase {
9595
reducer: effectsCancellationReducer,
9696
environment: .init(
9797
mainQueue: self.scheduler.eraseToAnyScheduler(),
98-
trivia: { n in Effect(value: "\(n) is a good number Brent") }
98+
numberFact: { n in Effect(value: "\(n) is a good number Brent") }
9999
)
100100
)
101101

Sources/ComposableArchitecture/SwiftUI/WithViewStore.swift

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,6 @@ extension WithViewStore: DynamicViewContent where State: Collection, Content: Dy
111111
/// from store state.
112112
@available(iOS 14, macOS 11, tvOS 14, watchOS 7, *)
113113
extension WithViewStore: Scene where Content: Scene {
114-
public typealias Body = Content
115-
116114
/// Initializes a structure that transforms a store into an observable view store in order to
117115
/// compute scenes from store state.
118116

0 commit comments

Comments
 (0)