Skip to content

Commit

Permalink
[WEB-749] ReactiveExtensions and ReactiveSwift to SPM (#1745)
Browse files Browse the repository at this point in the history
* after adding a package.swift file to ReactiveExtensions, imported through SPM, included ReactiveSwift as a dependency. Almost build except for a few import ReactiveExtensions (could not find) build errors.

* project building with Foundation and UIKit imports

* Imported ReactiveSwift via SPM to correct linker issues.

* removed unneeded dependencies from Library-iOS and KickstarterFramework-iOS

Because KsApi includes ReactiveSwift and ReactiveExtensions, and Library includes KsApi and KickstarterFramework-iOS includes Library-iOS, there is no need to include RE and RS in either Library-iOS or KickstarterFramework-iOS.

* formatting

* Found that including RE and RS in Library-iOS and only RE in KF-iOS required for on device build.

* Removed unneeded ReactiveExtensions-TestHelpers from KsApi
  • Loading branch information
msadoon committed Oct 19, 2022
1 parent 63e5d5e commit bb76488
Show file tree
Hide file tree
Showing 50 changed files with 167 additions and 185 deletions.
4 changes: 0 additions & 4 deletions Cartfile
@@ -1,7 +1,3 @@
### Internal

github "kickstarter/Kickstarter-ReactiveExtensions" "e3f7786b5bcc7b99c14b9fd313302bb59d9c3fe9"

### 3rd Party

github "ReactiveCocoa/ReactiveSwift" == 6.5.0
3 changes: 1 addition & 2 deletions Cartfile.resolved
@@ -1,2 +1 @@
github "ReactiveCocoa/ReactiveSwift" "6.5.0"
github "kickstarter/Kickstarter-ReactiveExtensions" "e3f7786b5bcc7b99c14b9fd313302bb59d9c3fe9"

3 changes: 0 additions & 3 deletions Carthage-xcfilelist/app-input-files.xcfilelist
@@ -1,4 +1 @@
# Carthage input files

$(SRCROOT)/Carthage/Build/iOS/ReactiveExtensions.framework
$(SRCROOT)/Carthage/Build/iOS/ReactiveSwift.framework
3 changes: 0 additions & 3 deletions Carthage-xcfilelist/app-output-files.xcfilelist
@@ -1,4 +1 @@
# Carthage output files

$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveExtensions.framework
$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveSwift.framework
@@ -1,4 +1 @@
# Carthage input files

$(SRCROOT)/Carthage/Build/iOS/ReactiveExtensions.framework
$(SRCROOT)/Carthage/Build/iOS/ReactiveSwift.framework
@@ -1,4 +1 @@
# Carthage output files

$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveExtensions.framework
$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveSwift.framework
3 changes: 0 additions & 3 deletions Carthage-xcfilelist/ksapi-input-files.xcfilelist
@@ -1,4 +1 @@
# Carthage input files

$(SRCROOT)/Carthage/Build/iOS/ReactiveExtensions.framework
$(SRCROOT)/Carthage/Build/iOS/ReactiveSwift.framework
3 changes: 0 additions & 3 deletions Carthage-xcfilelist/ksapi-output-files.xcfilelist
@@ -1,4 +1 @@
# Carthage output files

$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveExtensions.framework
$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveSwift.framework
3 changes: 0 additions & 3 deletions Carthage-xcfilelist/library-input-files.xcfilelist
@@ -1,4 +1 @@
# Carthage input files

$(SRCROOT)/Carthage/Build/iOS/ReactiveExtensions.framework
$(SRCROOT)/Carthage/Build/iOS/ReactiveSwift.framework
3 changes: 0 additions & 3 deletions Carthage-xcfilelist/library-output-files.xcfilelist
@@ -1,4 +1 @@
# Carthage output files

$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveExtensions.framework
$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/ReactiveSwift.framework
@@ -1,3 +1,4 @@
import Foundation
import Library
import Prelude
import ReactiveSwift
Expand Down
262 changes: 110 additions & 152 deletions Kickstarter.xcodeproj/project.pbxproj

Large diffs are not rendered by default.

Expand Up @@ -180,6 +180,15 @@
"revision" : "b0cec69c19a13088864b52ad1745397f587daec7"
}
},
{
"identity" : "kickstarter-reactiveextensions",
"kind" : "remoteSourceControl",
"location" : "https://github.com/kickstarter/Kickstarter-ReactiveExtensions",
"state" : {
"branch" : "feature/swift-package",
"revision" : "34276be97fa5acae36e714ad65f8ecb733c0566b"
}
},
{
"identity" : "kingfisher",
"kind" : "remoteSourceControl",
Expand Down Expand Up @@ -243,6 +252,15 @@
"version" : "1.16.3"
}
},
{
"identity" : "reactiveswift",
"kind" : "remoteSourceControl",
"location" : "https://github.com/ReactiveCocoa/ReactiveSwift",
"state" : {
"revision" : "e03cda84105ba707de039b757e2b2de868c65a3e",
"version" : "6.5.0"
}
},
{
"identity" : "sdwebimage",
"kind" : "remoteSourceControl",
Expand Down
2 changes: 1 addition & 1 deletion KsApi/models/Project.swift
@@ -1,4 +1,4 @@

import Foundation
import Prelude
import ReactiveSwift

Expand Down
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public protocol ActivitySampleBackingCellViewModelInputs {
/// Call to configure cell with activity value.
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/ActivitySampleFollowCellViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/AddNewCardViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
2 changes: 1 addition & 1 deletion Library/ViewModels/BetaToolsViewModel.swift
@@ -1,8 +1,8 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public enum BetaToolsRow: Int, CaseIterable {
case debugConfigFeatureFlags
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/CommentCellViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveSwift
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/CommentRepliesViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/CommentsViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/DashboardFundingCellViewModel.swift
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public struct FundingGraphData {
public let project: Project
Expand Down
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public protocol DashboardReferrerRowStackViewViewModelInputs {
/// Call to configure cell with referrer data.
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/DashboardVideoCellViewModel.swift
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public protocol DashboardVideoCellViewModelInputs {
/// Call to configure cell with video stats.
Expand Down
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public protocol DiscoveryNavigationHeaderViewModelInputs {
/// Call to configure with Discovery params.
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/DiscoveryPageViewModel.swift
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public protocol DiscoveryPageViewModelInputs {
/// Call when the Config has been updated in the AppEnvironment
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/DiscoveryViewModel.swift
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public protocol DiscoveryViewModelInputs {
/// Call when Recommendations setting changes on Settings > Account > Privacy > Recommendations.
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/ErroredBackingViewViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveSwift
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/FacebookConfirmationViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import ReactiveExtensions
import ReactiveSwift
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/LoginViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/MessageCellViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import ReactiveSwift

Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/MessageCellViewModelTests.swift
@@ -1,3 +1,4 @@
import Foundation
@testable import KsApi
import Library
import ReactiveExtensions_TestHelpers
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/MessageThreadCellViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/MessageThreadCellViewModelTests.swift
@@ -1,3 +1,4 @@
import Foundation
@testable import KsApi
import Library
import Prelude
Expand Down
2 changes: 1 addition & 1 deletion Library/ViewModels/PledgeAmountViewModel.swift
@@ -1,8 +1,8 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public typealias PledgeAmountData = (amount: Double, min: Double, max: Double, isValid: Bool)

Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/PledgeViewCTAContainerViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveSwift
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/ProjectPageViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveSwift
Expand Down
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveSwift
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/RootCommentCellViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveSwift
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/ThanksViewModel.swift
Expand Up @@ -2,6 +2,7 @@ import KsApi
import Prelude
import ReactiveExtensions
import ReactiveSwift
import UIKit

public typealias ThanksPageData = (
project: Project,
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/TwoFactorViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveExtensions
Expand Down
1 change: 1 addition & 0 deletions Library/ViewModels/WatchProjectViewModel.swift
@@ -1,3 +1,4 @@
import Foundation
import KsApi
import Prelude
import ReactiveSwift
Expand Down

0 comments on commit bb76488

Please sign in to comment.