Merge pull request #5 from f-meloni/fix_execute_step

Fix executeStep method

Sources/RocketLib/Executors/CommitExecutor.swift

@@ -1,7 +1,7 @@
 import Logger
 
 final class CommitExecutor: ScriptLauncherExecutor<CommitParameters> {
-    func executeStep(version: String, logger: Logger) {
+    override func executeStep(version: String, logger: Logger) {
         let message = parameters.message ?? "Version \(version)"
         var scriptString = "git commit -m \"\(message)\""
 

Sources/RocketLib/Executors/DefaultExecutor.swift

@@ -6,4 +6,8 @@ class DefaultExecutor<T: StepParameters>: StepExecutor {
     required init(dictionary: [String: Any]?) {
         parameters = T(dictionary: dictionary)
     }
+
+    func executeStep(version _: String, logger _: Logger) {
+        fatalError("Not implemented")
+    }
 }

Sources/RocketLib/Executors/GitAddExecutor.swift

@@ -1,7 +1,7 @@
 import Logger
 
 final class GitAddExecutor: ScriptLauncherExecutor<GitAddParameters> {
-    func executeStep(version _: String, logger: Logger) {
+    override func executeStep(version _: String, logger: Logger) {
         launchScript(content: "git add " + parameters.paths.joined(separator: " "), errorMessage: "Git Add step failed with error", logger: logger)
     }
 }

Sources/RocketLib/Executors/HideDevDependenciesExecutor.swift

@@ -3,7 +3,7 @@ import Logger
 final class HideDevDependenciesExecutor: DefaultExecutor<ModifyDevDependenciesParameters> {
     var devDependenciesModifier: DevDependenciesModifing = DevDependenciesModifier()
 
-    func executeStep(version _: String, logger: Logger) {
+    override func executeStep(version _: String, logger: Logger) {
         do {
             try devDependenciesModifier.hideDependencies(packagePath: parameters.packagePath)
         } catch {

Sources/RocketLib/Executors/PushExecutor.swift

@@ -1,7 +1,7 @@
 import Logger
 
 final class PushExecutor: ScriptLauncherExecutor<PushParameters> {
-    func executeStep(version _: String, logger: Logger) {
+    override func executeStep(version _: String, logger: Logger) {
         var scriptString = "git push \(parameters.remote) \(parameters.branch) --tags"
 
         if parameters.noVerify {

Sources/RocketLib/Executors/ScriptExecutor.swift

@@ -2,7 +2,7 @@ import Foundation
 import Logger
 
 final class ScriptExecutor: ScriptLauncherExecutor<ScriptParameters> {
-    func executeStep(version _: String, logger: Logger) {
+    override func executeStep(version _: String, logger: Logger) {
         guard let content = parameters.content,
             !content.isEmpty else {
             logger.logError("Invalid script content")

Sources/RocketLib/Executors/StepExecutor.swift

@@ -6,7 +6,3 @@ public protocol StepExecutor {
 
     init(dictionary: [String: Any]?)
 }
-
-extension StepExecutor {
-    func executeStep(version _: String, logger _: Logger) {}
-}

Sources/RocketLib/Executors/SwiftScriptExecutor.swift

@@ -5,7 +5,7 @@ final class SwiftScriptExecutor: DefaultExecutor<SwiftScriptParameters> {
     lazy var fileManager: FileManager = .default
     lazy var processLauncher: ProcessLaunching = ProcessLauncher()
 
-    func executeStep(version: String, logger: Logger) {
+    override func executeStep(version: String, logger: Logger) {
         guard let scriptPath = parameters.scriptPath else {
             logger.logError("Invalid script path for the Swift script step")
             return

Sources/RocketLib/Executors/TagExecutor.swift

@@ -2,7 +2,7 @@ import Foundation
 import Logger
 
 final class TagExecutor: ScriptLauncherExecutor<TagParameters> {
-    func executeStep(version: String, logger: Logger) {
+    override func executeStep(version: String, logger: Logger) {
         launchScript(content: "git tag \(version)", errorMessage: "Tag step failed with error", logger: logger)
     }
 }

Sources/RocketLib/Executors/UnhideDevDependenciesExecutor.swift

@@ -3,7 +3,7 @@ import Logger
 final class UnhideDevDependenciesExecutor: DefaultExecutor<ModifyDevDependenciesParameters> {
     var devDependenciesModifier: DevDependenciesModifing = DevDependenciesModifier()
 
-    func executeStep(version _: String, logger: Logger) {
+    override func executeStep(version _: String, logger: Logger) {
         do {
             try devDependenciesModifier.unhideDependencies(packagePath: parameters.packagePath)
         } catch {