Alternative method of validationResult merge method

[nrikiji]

f9e0b59#diff-82cfdd33ac3981dee67efa99e2338154L14

this is question.

With this commit, ValidationResult.swift's merge method has been deleted, but I would like to know if there is an alternative method here

[piv199]

@adamwaite this seems reasonable.

As fo this commit I also can't understand why you have moved to ValidationError everywhere thus it gives additional overhead of passing non-owning errors?

[muni510]

f9e0b59#diff-82cfdd33ac3981dee67efa99e2338154L14

this is question.

With this commit, ValidationResult.swift's merge method has been deleted, but I would like to know if there is an alternative method here

I have extended the class with merge method

`
extension ValidationResult {
/**

 Merges the receiving validation rule with another.
 
 ```
 .valid.merge(.valid) // = .valid
 .valid.merge(.invalid([err])) // = .invalid([err])
 .invalid([err1]).merge(.invalid([err2]) // = .invalid([err1, err2])
 ```
 
 - Parameters:
    - result: The result to merge into the receiver.
 
 - Returns:
 Merged validation result.
 
 */
public func merge(with result: ValidationResult) -> ValidationResult {
    switch self {
    case .valid: return result
    case .invalid(let errorMessages):
        switch result {
        case .valid:
            return self
        case .invalid(let errorMessagesAnother):
            return .invalid([errorMessages, errorMessagesAnother].flatMap { $0 })
        }
    }
}

/**
 
 Merges the receiving validation rule with multiple others.
 
 - Parameters:
    - results: The results to merge the receiver.
 
 - Returns:
 Merged validation result.
 
 */
public func merge(with results: [ValidationResult]) -> ValidationResult {
    return results.reduce(self) { return $0.merge(with: $1) }
}

/**
 
 Merges multiple validation rules together.
 
 - Parameters:
    - results: The results to merge.
 
 - Returns:
 Merged validation result.
 
 */
public static func merge(results: [ValidationResult]) -> ValidationResult {
    return ValidationResult.valid.merge(with: results)
}

}`