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 most python2 compatibility #88

Merged
merged 5 commits into from
Mar 16, 2024
Merged

Conversation

a-detiste
Copy link
Contributor

I'm also trying to please pyflakes to see where real errors are.

@cdent cdent merged commit ad26e1c into pasteorg:master Mar 16, 2024
6 checks passed
@cdent
Copy link
Collaborator

cdent commented Mar 16, 2024

Thanks, again, very much.

@a-detiste
Copy link
Contributor Author

a-detiste commented Mar 16, 2024

There are still remains of six to clean up: six.ensure_text( & six.ensure_binary( are especialy hard to fix usually. I'll have a look at it later. In the meantime 3.8.0 is packaged for Debian. Greetings

@a-detiste
Copy link
Contributor Author

A bit of typing annotations would help .... I will see

@cdent
Copy link
Collaborator

cdent commented Mar 16, 2024

A bit of typing annotations would help .... I will see

I'd prefer we not put typing annotations into Paste.

Doing so implies a commitment to Paste longevity, and I don't think we want to do that. At this stage in its lifecycle paste should be used by people who have to for maintaining existing (and old) projects. It should never be chosen for something new.

@a-detiste
Copy link
Contributor Author

Ok I respect your point of view. I'll keep it simple.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants