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

Replace deprecated io/ioutil with io and os #1684

Merged
merged 1 commit into from
Dec 21, 2024
Merged

Replace deprecated io/ioutil with io and os #1684

merged 1 commit into from
Dec 21, 2024

Conversation

alexandear
Copy link
Contributor

@alexandear alexandear commented Dec 10, 2024 •
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 #1469 #1478

brackendawson
brackendawson approved these changes Dec 14, 2024
View reviewed changes
Copy link
Collaborator

@brackendawson brackendawson left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, allowing time for others to comment as usual.

@brackendawson brackendawson merged commit 7c367bb into stretchr:master Dec 21, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ccoVeille ccoVeille ccoVeille approved these changes

@brackendawson brackendawson brackendawson approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alexandear @brackendawson @ccoVeille