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.

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

refactor: move from io/ioutil to io and os packages #10

Closed
wants to merge 2 commits into from
Closed

refactor: move from io/ioutil to io and os packages #10

wants to merge 2 commits into from

Conversation

Juneezee
Copy link

@Juneezee Juneezee commented Jan 5, 2022

This PR introduces two small changes:

  1. Run go mod tidy
  2. The io/ioutil package has been deprecated in Go 1.16 (See https://golang.org/doc/go1.16#ioutil). This PR replaces the existing io/ioutil functions with their new definitions in io and os packages.

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
The io/ioutil package has been deprecated as of Go 1.16, see
https://golang.org/doc/go1.16#ioutil. This commit replaces the existing
io/ioutil functions with their new definitions in io and os packages.

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
@Becivells Becivells added the good first issue Good for newcomers label Jan 5, 2022
Copy link
Member

@Becivells Becivells left a comment

Choose a reason for hiding this comment

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

很抱歉,看了您的 github 我们不太赞成您类似刷 pr 的行为。目前版本还可以使用。这些不会影响您的使用,未来如果golang有明确的放弃这些packages 的行为。我们会及时处理,如果您喜欢 fofax,欢迎提供功能改进和使用问题

@Becivells Becivells closed this Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants