public -> open for subclassing

Kommander.podspec

@@ -1,12 +1,12 @@
 Pod::Spec.new do |s|
   s.name             = 'Kommander'
-  s.version          = '0.2.0'
+  s.version          = '0.2.1'
   s.summary          = 'A command pattern implementation written in Swift 3'
 
-  s.homepage         = 'https://github.com/intelygenz/Kommander-iOS'
+  s.homepage         = 'https://gitlab.intelygenz.com/ios/kommander'
   s.license          = { :type => 'MIT', :file => 'LICENSE' }
   s.author           = { 'Alex Rupérez' => '[email protected]', 'Juan Trías' => '[email protected]', 'Roberto Estrada' => '[email protected]' }
-  s.source           = { :git => 'https://github.com/intelygenz/Kommander-iOS.git', :tag => s.version.to_s }
+  s.source           = { :git => 'https://gitlab.intelygenz.com/ios/kommander.git', :tag => s.version.to_s }
 
   s.ios.deployment_target = '8.0'
 

Kommander.xcodeproj/project.pbxproj

@@ -348,8 +348,8 @@
 				CODE_SIGN_IDENTITY = "";
 				DEFINES_MODULE = YES;
 				DEVELOPMENT_TEAM = 3VW789WSMP;
-				DYLIB_COMPATIBILITY_VERSION = 1;
-				DYLIB_CURRENT_VERSION = 1;
+				DYLIB_COMPATIBILITY_VERSION = 0.2.0;
+				DYLIB_CURRENT_VERSION = 0.2.1;
 				DYLIB_INSTALL_NAME_BASE = "@rpath";
 				INFOPLIST_FILE = Kommander/Info.plist;
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
@@ -369,8 +369,8 @@
 				CODE_SIGN_IDENTITY = "";
 				DEFINES_MODULE = YES;
 				DEVELOPMENT_TEAM = 3VW789WSMP;
-				DYLIB_COMPATIBILITY_VERSION = 1;
-				DYLIB_CURRENT_VERSION = 1;
+				DYLIB_COMPATIBILITY_VERSION = 0.2.0;
+				DYLIB_CURRENT_VERSION = 0.2.1;
 				DYLIB_INSTALL_NAME_BASE = "@rpath";
 				INFOPLIST_FILE = Kommander/Info.plist;
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";

Kommander/CurrentDispatcher.swift

@@ -8,7 +8,7 @@
 
 import Foundation
 
-public class CurrentDispatcher: MainDispatcher {
+open class CurrentDispatcher: MainDispatcher {
 
     public override init() {
         super.init()

Kommander/Dispatcher.swift

@@ -12,7 +12,7 @@ private enum Priority {
     case operation, dispatch
 }
 
-public class Dispatcher {
+open class Dispatcher {
 
     internal final var operationQueue = OperationQueue()
     internal final var dispatchQueue = DispatchQueue(label: UUID().uuidString)
@@ -33,15 +33,15 @@ public class Dispatcher {
         priority = .dispatch
     }
 
-    public func execute(_ operation: Operation) {
+    open func execute(_ operation: Operation) {
         operationQueue.addOperation(operation)
     }
 
-    public func execute(_ operations: [Operation], waitUntilFinished: Bool = false) {
+    open func execute(_ operations: [Operation], waitUntilFinished: Bool = false) {
         operationQueue.addOperations(operations, waitUntilFinished: waitUntilFinished)
     }
 
-    public func execute(_ block: @escaping () -> Void) -> Any {
+    open func execute(_ block: @escaping () -> Void) -> Any {
         if priority == .dispatch {
             return execute(qos: nil, flags: nil, block: block)
         }
@@ -52,7 +52,7 @@ public class Dispatcher {
         }
     }
 
-    public func execute(_ blocks: [() -> Void], concurrent: Bool = true, waitUntilFinished: Bool = false) -> [Any] {
+    open func execute(_ blocks: [() -> Void], concurrent: Bool = true, waitUntilFinished: Bool = false) -> [Any] {
         var actions = [Any]()
         if concurrent {
             for block in blocks {
@@ -74,13 +74,13 @@ public class Dispatcher {
         return actions
     }
 
-    public func execute(qos: DispatchQoS?, flags: DispatchWorkItemFlags?, block: @escaping @convention(block) () -> ()) -> DispatchWorkItem {
+    open func execute(qos: DispatchQoS?, flags: DispatchWorkItemFlags?, block: @escaping @convention(block) () -> ()) -> DispatchWorkItem {
         let work = DispatchWorkItem(qos: qos ?? .default, flags: flags ?? .assignCurrentContext, block: block)
         execute(work)
         return work
     }
 
-    public func execute(_ work: DispatchWorkItem) {
+    open func execute(_ work: DispatchWorkItem) {
         dispatchQueue.async(execute: work)
     }
 

Kommander/Info.plist

@@ -15,7 +15,7 @@
 	<key>CFBundlePackageType</key>
 	<string>FMWK</string>
 	<key>CFBundleShortVersionString</key>
-	<string>1.0</string>
+	<string>0.2.1</string>
 	<key>CFBundleVersion</key>
 	<string>$(CURRENT_PROJECT_VERSION)</string>
 	<key>NSPrincipalClass</key>

Kommander/Kommand.swift

@@ -8,7 +8,7 @@
 
 import Foundation
 
-public class Kommand<T> {
+open class Kommand<T> {
 
     public typealias ActionBlock = () throws -> T
     public typealias SuccessBlock = (_ result: T) -> Void
@@ -21,23 +21,23 @@ public class Kommand<T> {
     private(set) internal final var errorBlock: ErrorBlock?
     internal final var action: Any?
 
-    internal init(deliverer: Dispatcher, executor: Dispatcher, actionBlock: @escaping ActionBlock) {
+    public init(deliverer: Dispatcher, executor: Dispatcher, actionBlock: @escaping ActionBlock) {
         self.deliverer = deliverer
         self.executor = executor
         self.actionBlock = actionBlock
     }
 
-    public func onSuccess(_ onSuccess: @escaping SuccessBlock) -> Self {
+    open func onSuccess(_ onSuccess: @escaping SuccessBlock) -> Self {
         self.successBlock = onSuccess
         return self
     }
 
-    public func onError(_ onError: @escaping ErrorBlock) -> Self {
+    open func onError(_ onError: @escaping ErrorBlock) -> Self {
         self.errorBlock = onError
         return self
     }
 
-    public func execute() {
+    open func execute() {
         action = executor.execute {
             do {
                 let result = try self.actionBlock()
@@ -52,7 +52,7 @@ public class Kommand<T> {
         }
     }
 
-    public func cancel() {
+    open func cancel() {
         if let operation = action as? Operation, operation.isExecuting {
             operation.cancel()
         }

Kommander/Kommander.swift

@@ -8,7 +8,7 @@
 
 import Foundation
 
-public class Kommander {
+open class Kommander {
 
     private final let deliverer: Dispatcher
     private final let executor: Dispatcher
@@ -48,19 +48,19 @@ public class Kommander {
         executor = Dispatcher(label: name, qos: qos, attributes: attributes, autoreleaseFrequency: autoreleaseFrequency, target: target)
     }
 
-    public func makeKommand<T>(_ actionBlock: @escaping () throws -> T) -> Kommand<T> {
+    open func makeKommand<T>(_ actionBlock: @escaping () throws -> T) -> Kommand<T> {
         return Kommand<T>(deliverer: deliverer, executor: executor, actionBlock: actionBlock)
     }
 
-    public func makeKommands<T>(_ actionBlocks: [() throws -> T]) -> [Kommand<T>] {
+    open func makeKommands<T>(_ actionBlocks: [() throws -> T]) -> [Kommand<T>] {
         var kommands = [Kommand<T>]()
         for actionBlock in actionBlocks {
             kommands.append(Kommand<T>(deliverer: deliverer, executor: executor, actionBlock: actionBlock))
         }
         return kommands
     }
 
-    func execute<T>(_ kommands: [Kommand<T>], concurrent: Bool = true, waitUntilFinished: Bool = false) {
+    open func execute<T>(_ kommands: [Kommand<T>], concurrent: Bool = true, waitUntilFinished: Bool = false) {
         let blocks = kommands.map { kommand -> () -> Void in
             return {
                 do {
@@ -81,7 +81,7 @@ public class Kommander {
         }
     }
 
-    func cancel<T>(_ kommands: [Kommand<T>]) {
+    open func cancel<T>(_ kommands: [Kommand<T>]) {
         for kommand in kommands {
             kommand.cancel()
         }

Kommander/MainDispatcher.swift

@@ -8,7 +8,7 @@
 
 import Foundation
 
-public class MainDispatcher: Dispatcher {
+open class MainDispatcher: Dispatcher {
 
     public init() {
         super.init(name: nil, qos: nil, maxConcurrentOperationCount: OperationQueue.defaultMaxConcurrentOperationCount)