Skip to content

Conversation

@tristanmorgan
Copy link
Member

Backport of #451, replaces deprecated io/ioutil package usage.

@angrycub
Copy link
Contributor

Everyplace where we used ioutil.TempDir in tests could possibly be better served with testing's t.TempDir function, since it will clean them up automatically.

@tristanmorgan
Copy link
Member Author

Hi @angrycub, I have updated the branch to change tests to use t.TempDir() now.

@crw
Copy link
Contributor

crw commented May 14, 2025

@tristanmorgan do you need help getting this reviewed? I registered this as an internal contributor change, which usually has its own workflow.

It can be difficult to get changes reviewed in go-getter, due to the reluctance to disrupt anything for any of the products that use it. I'm assuming this change is with regards to a specific need. Let me know.

@tristanmorgan
Copy link
Member Author

if #549 is merged this one will be redundant.

@YakDriver
Copy link
Member

Fixed by #549

@YakDriver YakDriver closed this Sep 5, 2025
@tristanmorgan tristanmorgan deleted the backport branch September 7, 2025 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants