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

fix: should not pass nil to context #21

Merged

xakep666 merged 2 commits into xakep666:master from taako-502:fix-warning-context-nil

Mar 25, 2024

Contributor

taako-502 commented Feb 12, 2024

I fixed the warning. According to the page below, we should use context.TODO() instead of nil.

https://pkg.go.dev/context#TODO

TODO returns a non-nil, empty Context. Code should use context.TODO when it's unclear which Context to use or it is not yet available (because the surrounding function has not yet been extended to accept a Context parameter).

taako-502 added 2 commits

February 12, 2024 08:36


          fix: should not pass nil to context

e71bc31


          delete blank

d5472b1

xakep666 merged commit 7bc6412 into xakep666:master

2 checks passed

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