Merge pull request #177 from RxSwiftCommunity/release/2-12-3

project: release 2.12.3

Author: twittemb

CHANGELOG.md

@@ -1,3 +1,9 @@
+** Version 2.12.3 **:
+
+- fix "Unhandled files" warnings in the Package.swift file
+- fix re-entrancy issue in the FlowCoordinator file
+- revert to a strong retain policy in the Reactive+UIViewController file (see version 2.12.0)
+
 ** Version 2.12.2 **:
 
 - ensure the navigate function is called on the main thread (regression introduced in 2.12.1)

Cartfile.resolved

@@ -1 +1 @@
-github "ReactiveX/RxSwift" "6.0.0"
+github "ReactiveX/RxSwift" "6.2.0"

Package.swift

@@ -1,4 +1,4 @@
-// swift-tools-version:5.3
+// swift-tools-version:5.4
 
 import PackageDescription
 
@@ -18,7 +18,8 @@ let package = Package(
                 "RxSwift",
                 .product(name: "RxCocoa", package: "RxSwift")
             ],
-            path: "RxFlow"
+            path: "RxFlow",
+            exclude: ["Info.plist"]
         ),
         .testTarget(
             name: "RxFlowTests",
@@ -27,7 +28,8 @@ let package = Package(
                 .product(name: "RxBlocking", package: "RxSwift"),
                 .product(name: "RxTest", package: "RxSwift")
             ],
-            path: "RxFlowTests"
+            path: "RxFlowTests",
+            exclude: ["Info.plist"]
         ),
     ],
     swiftLanguageVersions: [.v5]

RxFlow.podspec

@@ -1,7 +1,7 @@
 Pod::Spec.new do |s|
   s.name = "RxFlow"
-  s.version = "2.12.2"
-  s.swift_version = '5.3'
+  s.version = "2.12.3"
+  s.swift_version = '5.4'
   s.summary = "RxFlow is a navigation framework for iOS applications, based on a Reactive Coordinator pattern."
 
   s.description = <<-DESC

RxFlow.xcodeproj/project.pbxproj

@@ -288,7 +288,7 @@
 			isa = PBXProject;
 			attributes = {
 				LastSwiftUpdateCheck = 0920;
-				LastUpgradeCheck = 1200;
+				LastUpgradeCheck = 1250;
 				ORGANIZATIONNAME = RxSwiftCommunity;
 				TargetAttributes = {
 					1A8FBE741FF9783100389464 = {
@@ -604,7 +604,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
-				MARKETING_VERSION = 2.12.2;
+				MARKETING_VERSION = 2.12.3;
 				PRODUCT_BUNDLE_IDENTIFIER = io.warpfactor.RxFlow;
 				PRODUCT_NAME = "$(TARGET_NAME:c99extidentifier)";
 				PROVISIONING_PROFILE_SPECIFIER = "";
@@ -639,7 +639,7 @@
 					"@executable_path/Frameworks",
 					"@loader_path/Frameworks",
 				);
-				MARKETING_VERSION = 2.12.2;
+				MARKETING_VERSION = 2.12.3;
 				PRODUCT_BUNDLE_IDENTIFIER = io.warpfactor.RxFlow;
 				PRODUCT_NAME = "$(TARGET_NAME:c99extidentifier)";
 				PROVISIONING_PROFILE_SPECIFIER = "";

RxFlow.xcodeproj/xcshareddata/xcschemes/RxFlow.xcscheme

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <Scheme
-   LastUpgradeVersion = "1200"
+   LastUpgradeVersion = "1250"
    version = "1.3">
    <BuildAction
       parallelizeBuildables = "YES"

RxFlow/Extensions/Reactive+UIViewController.swift

@@ -17,7 +17,7 @@ public extension Reactive where Base: UIViewController {
     /// Rx observable, triggered when the view is being dismissed
     var dismissed: ControlEvent<Bool> {
         let dismissedSource = self.sentMessage(#selector(Base.viewDidDisappear))
-            .filter { [weak base] _ in base?.isBeingDismissed ?? true }
+            .filter { [base] _ in base.isBeingDismissed }
             .map { _ in false }
 
         let movedToParentSource = self.sentMessage(#selector(Base.didMove))

RxFlow/FlowCoordinator.swift

@@ -46,6 +46,7 @@ public final class FlowCoordinator: NSObject {
         // listen for the internal steps relay that aggregates the flow's Stepper's steps and
         // the FlowContributors's Stepper's steps
         self.stepsRelay
+            .observe(on: CurrentThreadScheduler.instance)
             .do(onDispose: { [weak self] in
                 self?.childFlowCoordinators.removeAll()
                 self?.parentFlowCoordinator?.childFlowCoordinators.removeValue(forKey: self?.identifier ?? "")
@@ -127,7 +128,7 @@ public final class FlowCoordinator: NSObject {
     private func performSideEffects(with flowContributor: FlowContributor) {
         switch flowContributor {
         case let .forwardToCurrentFlow(step):
-            stepsRelay.accept(step)
+            self.stepsRelay.accept(step)
         case let .forwardToParentFlow(step):
             parentFlowCoordinator?.stepsRelay.accept(step)
         case .contribute:

RxFlowDemo/Cartfile

@@ -1,2 +1,2 @@
 github "ReactiveX/RxSwift" ~> 6.0.0
-github "AliSoftware/Reusable" ~> 4.0.5
+github "AliSoftware/Reusable" ~> 4.1.1

RxFlowDemo/Cartfile.resolved

@@ -1,2 +1,2 @@
 github "AliSoftware/Reusable" "4.1.1"
-github "ReactiveX/RxSwift" "6.0.0"
+github "ReactiveX/RxSwift" "6.2.0"