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

test: use strings.ReplaceAll, remove unnecessary else #1542

Merged

erikdubbelboer merged 1 commit into valyala:master from alexandear:refactor-http-test

Apr 25, 2023

Contributor

alexandear commented Apr 25, 2023

This PR does the following refactorings in tests:

use strings.ReplaceAll(...) instead of strings.Replace(..., -1);
remove else.


          test: use ReplaceAll, remove unnecessary else

e1da8fa

erikdubbelboer merged commit 1aac293 into valyala:master

Collaborator

erikdubbelboer commented Apr 25, 2023

Thanks!

alexandear deleted the refactor-http-test branch

April 25, 2023 11:47

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

Labels

None yet