Fixed code review issue

Davarg · Davarg · commit 0564ec899421 · 2025-04-15T10:06:52.000+03:00
diff --git a/Sources/DangerShellExecutor/ShellExecutor.swift b/Sources/DangerShellExecutor/ShellExecutor.swift
@@ -77,8 +77,8 @@ public struct ShellExecutor: ShellExecuting {
 
             task.waitUntilExit()
 
-            let result = String(data: data, encoding: .utf8) ?? ""
-            return result.trimmingCharacters(in: .whitespacesAndNewlines)
+            let result = String(data: data, encoding: .utf8).map { $0.trimmingCharacters(in: .whitespacesAndNewlines) } ?? ""
+            return result
         } catch {
             return error.localizedDescription
         }

Original file line number	Diff line number	Diff line change
`@@ -77,8 +77,8 @@ public struct ShellExecutor: ShellExecuting {`
`77`	`77`
`78`	`78`	`task.waitUntilExit()`
`79`	`79`
`80`		`- let result = String(data: data, encoding: .utf8) ?? ""`
`81`		`- return result.trimmingCharacters(in: .whitespacesAndNewlines)`
	`80`	`+ let result = String(data: data, encoding: .utf8).map { $0.trimmingCharacters(in: .whitespacesAndNewlines) } ?? ""`
	`81`	`+ return result`
`82`	`82`	`} catch {`
`83`	`83`	`return error.localizedDescription`
`84`	`84`	`}`