New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(feg): replace deprecated errors module #12726
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
88b55d9
to
250158d
Compare
26bf127
to
fe8d4ba
Compare
a707103
to
29d87eb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes look good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the resulting log look pretty much the same? Do you have a diff? (nvm I saw the attached issue :) )
29d87eb
to
c2d2f2f
Compare
b64a540
to
72a6a7b
Compare
Replace functions from the github.com/pgk/errors package with `fmt.Errorf` in /magma/feg. Created in pairing with Moritz Huebner. Signed-off-by: Sebastian Wolf <sebastian.wolf@tngtech.com>
72a6a7b
to
7ed0baa
Compare
Replace functions from the github.com/pgk/errors package with `fmt.Errorf` in /magma/feg. Created in pairing with Moritz Huebner. Signed-off-by: Sebastian Wolf <sebastian.wolf@tngtech.com>
Replace functions
errors.Wrap
anderrors.Wrapf
from thegithub.com/pgk/errors package with
fmt.Errorsf
. Created in pairingwith Moritz Huebner.
Signed-off-by: Sebastian Wolf sebastian.wolf@tngtech.com
Summary
This PR implements the proposed solutions of ticket #12632 for the feg module.
Test Plan
Perform unit tests
Additional Information
Worked in pairing with @MoritzThomasHuebner