search

pelletier / go-toml

Go library for the TOML file format
https://github.com/pelletier/go-toml
Other
1.67k stars 208 forks source link

remove io/ioutil #910

Closed mpldr closed 8 months ago

mpldr commented 8 months ago

Since ioutil has been deprecated, replace the calls to ioutil with the functions that are now called anyways.

pelletier commented 8 months ago

Thank you for taking the time to make this change! I'm not keen on removing those deprecated imports yet, see https://github.com/pelletier/go-toml/issues/898#issuecomment-1716315155. Essentially I think changing those imports will be worth it either in case of a new major version of go-toml, or when Go actually removes them, whichever comes first.

mpldr commented 8 months ago

Ironically, this is now done in #911 anyways :D

pelletier commented 8 months ago

I have no problem updating the imports in the program that generates tests :)

mpldr commented 8 months ago

I mean, only one of them was in an imported package anyways. The rest was in tests or main packages.