[errors] Remove Fix(), move MultiErrorFromErrors().

[errors] Remove Fix(), move MultiErrorFromErrors().

Fix is no longer used by anything, and MultiErrorFromErrors is
purpose-built for the common/sync/parallel package, so it's a better fit
there.

R=dnj@chromium.org, vadimsh@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2958433002
Committed: https://github.com/luci/luci-go/commit/b2857583f8981670a82a1f52a249d93760b96de1

[Vadim Sh., 2017-06-23 00:32:16]

lgtm

https://codereview.chromium.org/2958433002/diff/1/common/sync/parallel/workPo...
File common/sync/parallel/workPool.go (right):

https://codereview.chromium.org/2958433002/diff/1/common/sync/parallel/workPo...
common/sync/parallel/workPool.go:41: func MultiErrorFromErrors(ch <-chan error)
error {
does it have to be public? Anyone uses it besides WorkPool?

[iannucci, 2017-06-23 00:38:26]

done

[iannucci, 2017-06-23 00:38:31]

The CQ bit was checked by iannucci@chromium.org

[iannucci, 2017-06-23 00:38:31]

The patchset sent to the CQ was uploaded after l-g-t-m from vadimsh@chromium.org
Link to the patchset: https://codereview.chromium.org/2958433002/#ps40001
(title: "keep it secret, keep it safe")

[commit-bot: I haz the power, 2017-06-23 00:38:43]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-23 00:55:34]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1498178311320580,
"parent_rev": "34a3b4f2ab59c6082472e3d880843ba11d3c7243", "commit_rev":
"b2857583f8981670a82a1f52a249d93760b96de1"}

[commit-bot: I haz the power, 2017-06-23 00:55:40]

Description was changed from

==========
[errors] Remove Fix(), move MultiErrorFromErrors().

Fix is no longer used by anything, and MultiErrorFromErrors is
purpose-built for the common/sync/parallel package, so it's a better fit
there.

R=dnj@chromium.org, vadimsh@chromium.org
BUG=
==========

to

==========
[errors] Remove Fix(), move MultiErrorFromErrors().

Fix is no longer used by anything, and MultiErrorFromErrors is
purpose-built for the common/sync/parallel package, so it's a better fit
there.

R=dnj@chromium.org, vadimsh@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2958433002
Committed:
https://github.com/luci/luci-go/commit/b2857583f8981670a82a1f52a249d93760b96de1
==========

[commit-bot: I haz the power, 2017-06-23 00:55:41]

Committed patchset #3 (id:40001) as
https://github.com/luci/luci-go/commit/b2857583f8981670a82a1f52a249d93760b96de1