Minor whitespace cleanups, make some of the String usage a little more flexible (via StringProtocol)

Author: gwynne

Sources/Argument.swift

@@ -1,27 +1,24 @@
 import Foundation
 
-
 public enum Argument: Equatable {
     case quoted(QuotedString)
     case verbatim(String)
 
-    public init(quoted string: String) {
-        self = .quoted(.init(string))
+    public init(quoted string: some StringProtocol) {
+        self = .quoted(.init(.init(string)))
     }
 
-    public init(verbatim string: String) {
-        self = .verbatim(string)
+    public init(verbatim string: some StringProtocol) {
+        self = .verbatim(.init(string))
     }
 }
 
-
 extension Argument: ExpressibleByStringLiteral {
     public init(stringLiteral value: StringLiteralType) {
         self = .quoted(.init(value))
     }
 }
 
-
 extension Argument: CustomStringConvertible {
     public var description: String {
         switch self {
@@ -33,19 +30,17 @@ extension Argument: CustomStringConvertible {
     }
 }
 
-
 extension Argument {
     public static func url(_ url: URL) -> Self { url.absoluteString.verbatim }
 }
 
 
-extension String {
+extension StringProtocol {
     public var quoted: Argument { .init(quoted: self) }
     public var verbatim: Argument { .init(verbatim: self) }
 }
 
-
-extension Array where Element == String {
+extension Sequence<StringProtocol> {
     public var quoted: [Argument] { map(\.quoted) }
     public var verbatim: [Argument] { map(\.verbatim) }
 }

Sources/QuotedString.swift

@@ -1,6 +1,5 @@
 import ShellQuote
 
-
 public struct QuotedString: Equatable {
     public var unquoted: String
     public var quoted: String
@@ -11,7 +10,6 @@ public struct QuotedString: Equatable {
     }
 }
 
-
 extension QuotedString: CustomStringConvertible {
     public var description: String { quoted }
 }

Sources/ShellOutCommand+other.swift

@@ -15,7 +15,7 @@ public extension ShellOutCommand {
 
     /// Create a file with a given name and contents (will overwrite any existing file with the same name)
     static func createFile(named name: String, contents: String) -> ShellOutCommand {
-        .bash(arguments: ["-c", #"echo \#(contents.quoted) > \#(name.quoted)"#.verbatim])
+        .bash(arguments: ["-c", "echo \(contents.quoted) > \(name.quoted)".verbatim])
     }
 
     /// Move a file from one path to another

Sources/ShellOutError.swift

@@ -35,12 +35,6 @@ extension ShellOutError: LocalizedError {
 
 private extension Data {
     func shellOutput() -> String {
-        let output = String(decoding: self, as: UTF8.self)
-
-        guard !output.hasSuffix("\n") else {
-            return String(output.dropLast())
-        }
-
-        return output
+        .init(String(decoding: self, as: UTF8.self).trimmingSuffix(while: { $0.isNewline }))
     }
 }