[Review, 2nd] SF-0007: Subprocess

[icharleshu (Charles Hu)]

@davedelong

Instead, it would be nice to have a Subprocess.Configuration struct that encapsulates all those parameters, so that the .run and .runDetached methods just take those.

Subprocess already supports configuration based run:

public static func run<R>(
    _ configuration: Configuration,
    output: RedirectedOutputMethod = .redirectToSequence,
    error: RedirectedOutputMethod = .redirectToSequence,
     _ body: (sending @escaping (Subprocess, StandardInputWriter) async throws -> R)
) async throws -> ExecutionResult<R>

I'll add an overload for runDetached as well.