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 #20

Closed

taako-502 wants to merge 2 commits into xakep666:master from taako-502:fix-warning

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 11, 2024 21:50


          version up

0bc0c7d


          fix: should not pass nil to context

c2a7b6b

taako-502 closed this

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

Labels

None yet