Skip to content

Replace io/ioutil with io and os #1469

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
Closed
alexandear wants to merge 1 commit into from
Closed

Replace io/ioutil with io and os #1469

alexandear wants to merge 1 commit into from

Conversation

@alexandear
Copy link
Contributor

@alexandear alexandear commented Sep 12, 2023 •
edited
Loading

Summary

This PR replaces function usages from io/ioutil with the same functions from io and os.

Changes

  • replace ioutil.ReadFile with os.ReadFile
  • replace ioutil.ReadAll with io.ReadAll

Motivation

io/ioutil is deprecated:

Deprecated: As of Go 1.16, the same functionality is now provided by package io or package os, and those implementations should be preferred in new code. See the specific function documentation for details.

Related issues

#1305 #1386 #1407

@dolmen dolmen added the internal/refactor Refactor internals with no external visible changes label Sep 25, 2023
@dolmen
Copy link
Collaborator

dolmen commented Oct 10, 2023

Duplicate of #1407.

@dolmen dolmen closed this Oct 10, 2023
@alexandear
Copy link
Contributor Author

alexandear commented Oct 10, 2023

@dolmen #1407 is unmerged and closed. That's why I created the PR.

Also, #1407's changes are not full. We need to modify two files with ioutil import but #1407 PR modifies only one.

@alexandear alexandear mentioned this pull request Dec 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

internal/refactor Refactor internals with no external visible changes rejected/duplicate

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alexandear @dolmen