Skip to content

Remove use_2to3#5

Open
torbsorb wants to merge 1 commit intojimo1001:masterfrom
torbsorb:ISSUE-4-remove-use_2to3
Open

Remove use_2to3#5
torbsorb wants to merge 1 commit intojimo1001:masterfrom
torbsorb:ISSUE-4-remove-use_2to3

Conversation

@torbsorb
Copy link
Copy Markdown

@torbsorb torbsorb commented Sep 7, 2021

use_2to3 is no longer supported by setuptools. Let's deprecate Python 2
support, and remove the types unicode and long.

Closes 4

@torbsorb torbsorb force-pushed the ISSUE-4-remove-use_2to3 branch from bbb5712 to 843c701 Compare September 7, 2021 13:33
setup.py Outdated
rst2ast = rst2ast.cmd:run
""",
use_2to3 = True
"""
Copy link
Copy Markdown

@graingert graingert Sep 9, 2021

Choose a reason for hiding this comment

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

Suggested change
"""
""",
python_requires=">=3.6",

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks @graingert! Will fixup my commit

Copy link
Copy Markdown

@graingert graingert left a comment

Choose a reason for hiding this comment

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

you'll need to enable python_requires or people with python2 will accidentally install your version without py2 support

use_2to3 is no longer supported by setuptools. Let's deprecate Python 2
support, and remove the types `unicode` and `long`.

@graingert suggested adding `python_requires`, thanks!

Closes jimo1001#4
@torbsorb torbsorb force-pushed the ISSUE-4-remove-use_2to3 branch from 843c701 to f06dccb Compare September 13, 2021 06:53
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