Skip to content
This repository was archived by the owner on Feb 13, 2026. It is now read-only.

Comments

fix: remove six#115

Merged
busunkim96 merged 1 commit intomainfrom
remove-six
Sep 20, 2021
Merged

fix: remove six#115
busunkim96 merged 1 commit intomainfrom
remove-six

Conversation

@busunkim96
Copy link
Contributor

Remove six usage in this library. Six was used only for Python 2/3 compatibility.

Fixes #83, #84, #85, #86, #87, #88 🦕

@busunkim96 busunkim96 requested a review from a team as a code owner September 20, 2021 21:18
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 20, 2021
@product-auto-label product-auto-label bot added the api: dns Issues related to the googleapis/python-dns API. label Sep 20, 2021
@busunkim96 busunkim96 merged commit 95f94ef into main Sep 20, 2021
@busunkim96 busunkim96 deleted the remove-six branch September 20, 2021 21:31
This was referenced Sep 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: dns Issues related to the googleapis/python-dns API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests.unit.test__http.TestConnection: test_build_api_url_no_extra_query_params failed

2 participants