Skip to content
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

[Bug] ioutil is deprecated #17

Closed
1 of 3 tasks
vaguecoder opened this issue Jan 6, 2023 · 1 comment · Fixed by #24
Closed
1 of 3 tasks

[Bug] ioutil is deprecated #17

vaguecoder opened this issue Jan 6, 2023 · 1 comment · Fixed by #24
Assignees
@tanryberdi
Labels
bug Something isn't working golang Golang related code changes (required/expected) good first issue Good for newcomers misc Miscellaneous requests that don't fall under a feature, bug, documentation, etc.

Comments

@vaguecoder
Copy link
Owner

Describe the Bug

ioutil is deprecated. Replace all occurrences of ioutil with io or os w.r.t. the functions in use.

To Reproduce

NA

Additional Context

NA

Ownership of Contribution

If the triage is approved,

  • I wish to submit the PR myself.
  • I wish to submit the PR. But if someone is willing to do it, go ahead.
  • I don't know how to fix this or make these changes. I wish others to pick this up for me.
@vaguecoder vaguecoder added bug Something isn't working good first issue Good for newcomers misc Miscellaneous requests that don't fall under a feature, bug, documentation, etc. labels Jan 6, 2023
@vaguecoder vaguecoder mentioned this issue Jan 6, 2023
Merged
@tanryberdi
Copy link

@assign

@vaguecoder vaguecoder added the golang Golang related code changes (required/expected) label Jan 6, 2023
tanryberdi added a commit to tanryberdi/gorilla-mux that referenced this issue Jan 6, 2023
@tanryberdi
Refactor: ioutil to io
7953aa1 
- Changed ioutil package to io package

See vaguecoder#17
@tanryberdi tanryberdi mentioned this issue Jan 6, 2023
Merged
tanryberdi added a commit to tanryberdi/gorilla-mux that referenced this issue Jan 6, 2023
@tanryberdi
Refactor: ioutil to io
c39ff8f 
- Changed ioutil package to io package
Signed-off-by: Tanryberdi <tanryberdi@gmail.com>

See vaguecoder#17
tanryberdi added a commit to tanryberdi/gorilla-mux that referenced this issue Jan 7, 2023
@tanryberdi
Refactor: ioutil to io
dfbd902 
- Changed ioutil package to io package
Signed-off-by: Tanryberdi <tanryberdi@gmail.com>

See vaguecoder#17
vaguecoder pushed a commit that referenced this issue Jan 7, 2023
@tanryberdi @vaguecoder
Refactor: ioutil to io
66f6a2e 
- Changed ioutil package to io package
Signed-off-by: Tanryberdi <tanryberdi@gmail.com>

See #17
@vaguecoder vaguecoder linked a pull request Jan 7, 2023 that will close this issue
Refactor: ioutil to io #24
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tanryberdi tanryberdi

Labels
bug Something isn't working golang Golang related code changes (required/expected) good first issue Good for newcomers misc Miscellaneous requests that don't fall under a feature, bug, documentation, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor: ioutil to io tanryberdi/gorilla-mux
2 participants
@tanryberdi @vaguecoder