avalanchego
avalanchego copied to clipboard
ban usage of `require.Equal` when testing for errors
Why this should be merged
How this works
How this was tested
I feel like we shouldn't be merging this until (at least after) we enable the goerr113
linter. Currently - there are a number of parts of our code that do rely on error equality - and this may allow future incorrect changes to pass CI.
This PR has become stale because it has been open for 30 days with no activity. Adding the lifecycle/frozen
label will cause this PR to ignore lifecycle events.
This PR has become stale because it has been open for 30 days with no activity. Adding the lifecycle/frozen
label will cause this PR to ignore lifecycle events.
This PR has become stale because it has been open for 30 days with no activity. Adding the lifecycle/frozen
label will cause this PR to ignore lifecycle events.