Skip to content
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.

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

Remove use of six module #2036

Closed
3 tasks
drewj-tp opened this issue Dec 18, 2024 · 1 comment · Fixed by #2037
Closed
3 tasks

Remove use of six module #2036

drewj-tp opened this issue Dec 18, 2024 · 1 comment · Fixed by #2037
Assignees
Labels
cleanup Code/comment cleanup: Low Priority

Comments

@drewj-tp
Copy link
Contributor

The six module is great for projects that need to span py2 and py3. But that has not been the case for a long time.

Let's

  • drop if from our dependencies
  • replace instances of six.* with the py3 variants (e.g., six.string_types with str)
  • remove imports of six across the project
@drewj-tp drewj-tp added the cleanup Code/comment cleanup: Low Priority label Dec 18, 2024
@john-science
Copy link
Member

Oh, yeah, that can definitely go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Code/comment cleanup: Low Priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants