Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

remove io/util for advanced golang #2103

Merged
merged 1 commit into from
Nov 7, 2022
Merged

remove io/util for advanced golang #2103

merged 1 commit into from
Nov 7, 2022

Conversation

yanggangtony
Copy link
Contributor

We should clean all io/util package according the url: https://go.dev/doc/go1.16#ioutil

/kind cleanup

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2103/yanggangtony/remove-ioutil
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

Signed-off-by: yanggang <gang.yang@daocloud.io>
@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Nov 6, 2022
@yanggangtony
Copy link
Contributor Author

https://github.com/submariner-io/submariner/actions/runs/3404610115/jobs/5661968307
the ci test is wrong , but i think is not relevance with the pr..
😊

@sridhargaddam sridhargaddam merged commit 599654b into submariner-io:devel Nov 7, 2022
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2103/yanggangtony/remove-ioutil]

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ready-to-test When a PR is ready for full E2E testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants