approvals / ApprovalTests.Java

ApprovalTest verification library for Java
Apache License 2.0
336 stars 74 forks source link

New constructor for MultiScrubber that would take a collection of Scrubber #554

Closed murdos closed 1 week ago

murdos commented 2 weeks ago

Current constructor for MultiScrubber is not very easy to use.

Adding a new one could ease things:

public MultiScrubber(Collection<Scrubber> scrubbers) { 
  // ...
}

Or even:

public MultiScrubber(Scrubber... scrubbers) { 
  // ...
}
LarsEckart commented 1 week ago

Agreed, we made the change, will be in the next release.