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.

Sign up for GitHub

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

Jump to bottom

small fixes on ioutil and error handling #45

Merged

quasilyte merged 2 commits into quasilyte:master from peczenyj:fix-usage-of-deprecated-ioutil

Apr 18, 2024

Contributor

peczenyj commented Apr 18, 2024

Here I am

using io and os packages instead io/ioutil package - it is deprecated
using errors.New instead fmt.Errorf when there is no parameters to format
using %w instead %v when using fmt.Errort when wrapping an existing error
set unused errors in tests to _

enjoy

peczenyj added 2 commits

April 18, 2024 10:41


          stop use ioutil - this package is deprecated

df02cbc


          fix some error handling

4969c07

quasilyte approved these changes

View reviewed changes

Owner

quasilyte commented Apr 18, 2024

LGTM!

quasilyte merged commit b7ec239 into quasilyte:master

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet