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.

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

drop six dependency #250

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

drop six dependency #250

wants to merge 1 commit into from

Conversation

branchvincent
Copy link

Fixes #249

CoolCat467
CoolCat467 approved these changes Jan 17, 2024
View reviewed changes
Copy link

@CoolCat467 CoolCat467 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks pretty good to me as far as I can tell.

@c0fec0de
Copy link
Owner

Wanted to keep the py2.x support at the time then. But now its time to go on.

@coveralls
Copy link

Coverage Status

coverage: 99.914%. remained the same
when pulling c1ab78b on branchvincent:drop-six
into 65a5d09 on c0fec0de:main.

@a-detiste
Copy link

Yes please, this library landed today in Debian,
I'm glad this PR already exists.
https://wiki.debian.org/Python3-six-removal

@c0fec0de
Copy link
Owner

c0fec0de commented Jun 30, 2024 via email

@a-detiste
Copy link

No worry, it's a very long road.
image

@dminuoso
Copy link

August has passed, do you need more time or can I assist in getting this PR landed somehow?

Will take care until end of August. Super busy. Sorry for the delay. Alexandre Detiste @.> schrieb am Sa., 29. Juni 2024, 23:10:
…
Yes please, this library landed today in Debian, I'm glad this PR already exists. https://wiki.debian.org/Python3-six-removal — Reply to this email directly, view it on GitHub <#250 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE77IH57V3JS5ZQ25UK5LNDZJ4PEJAVCNFSM6AAAAABBFEDZ2GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJYGMZTMNJRGQ . You are receiving this because you commented.Message ID: @.>

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

@CoolCat467 CoolCat467 CoolCat467 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Drop six usage
6 participants
@branchvincent @c0fec0de @coveralls @a-detiste @dminuoso @CoolCat467