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

io/ioutil: flag package and functions with the Deprecated marker #43799

Closed

dlsniper wants to merge 1 commit into golang:master from dlsniper:dlsniper/deprecate-ioutil

Contributor

dlsniper commented Jan 20, 2021

All implementations are now part of io or os packages, per #42026.
Flag all implementations in ioutil, and the package itself, with
the Deprecated marker so that tooling can pick it up and interpret
it accordingly.

Updates #42026


          io/ioutil: flag package and functions with the Deprecated marker

f6ebc2a

All implementations are now part of io or os packages, per golang#42026.
Flag all implementations in ioutil, and the package itself, with
the Deprecated marker so that tooling can pick it up and interpret
it accordingly.

Updates golang#42026

google-cla bot added the cla: yes label

Contributor

gopherbot commented Jan 20, 2021

This PR (HEAD: f6ebc2a) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/go/+/284777 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

Contributor

gopherbot commented Jan 21, 2021

Message from Ian Lance Taylor:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Jan 21, 2021

Message from Florin Patan:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Jan 21, 2021

Message from Ian Lance Taylor:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Jun 2, 2021

Message from Colin Arnott:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Jun 2, 2021

Message from Russ Cox:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Nov 8, 2021

Message from Florin Patan:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Nov 8, 2021

Message from Ian Lance Taylor:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Nov 8, 2021

Message from Colin Arnott:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

gopherbot commented Nov 8, 2021

Message from Russ Cox:

Patch Set 1:

(1 comment)

Please don’t reply on this GitHub thread. Visit golang.org/cl/284777.
After addressing review feedback, remember to publish your drafts!

Contributor

jeffwidman commented Mar 24, 2022

@dlsniper do you want to rebase and fix merge conflicts per discussion over in #51927 (comment)?

This was referenced Mar 25, 2022

proposal: io/ioutil: Annotate functions with "Deprecated: " #51927

Closed

io/ioutil: add deprecation markers to ioutil #51961

Closed

Contributor

gopherbot commented Apr 7, 2024 •

edited by dmitshur

Loading

This PR is being closed because go.dev/cl/284777 has been abandoned.

Thanks for the CL. This change was applied a year later for Go 1.19 via CL 395918, so if this CL were to be rebased it'd become empty. Closing.

gopherbot closed this

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

Labels