Replace usage of go-multierror with errors package#16603
Conversation
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 6118 Click here to see the affected service packages
Action takenFound 19 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
|
None of these errors seem related in any way to the change- which affects test code in composer. I think just editing a global file (go.mod) kicked every test and there are numerous deviations kicking around. |
45e9cbe
This is still an indirect dependency through other packages, but gets rid of a direct pin and slightly modernises the relevant code.
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.