Skip to content

Add python_requires 3.6+ to setup.py#18

Merged
AbdealiLoKo merged 1 commit intocorridor:masterfrom
15jgme:Add_py_version_requirements
Nov 3, 2022
Merged

Add python_requires 3.6+ to setup.py#18
AbdealiLoKo merged 1 commit intocorridor:masterfrom
15jgme:Add_py_version_requirements

Conversation

@15jgme
Copy link
Copy Markdown
Contributor

@15jgme 15jgme commented Oct 8, 2022

For issue #10
Enforce a python version greater than 3.6 during install.
For pypi it may also be desirable to remove the following lines from setup.py as these versions will no longer be installable.
73 'Programming Language :: Python :: 2',
74 'Programming Language :: Python :: 2.7',
76 'Programming Language :: Python :: 3.4',
77 'Programming Language :: Python :: 3.5',

@indiVar0508
Copy link
Copy Markdown
Contributor

Hi @15jgme ,

yeah it makes sense can you also remove that in this commit itself

@15jgme
Copy link
Copy Markdown
Contributor Author

15jgme commented Oct 22, 2022

Hi @indiVar0508, for sure. Should be fixed now.

Going forward this library will only support py3.6+ and sqla1.4+

So, add python_requires in setuptools
@AbdealiLoKo AbdealiLoKo force-pushed the Add_py_version_requirements branch from 24962e7 to 6192cd3 Compare November 3, 2022 06:46
@AbdealiLoKo
Copy link
Copy Markdown
Contributor

Thanks a lot for the contribution @15jgme
I have made some small tweaks to your commit message and pushed
This should get merged soon

@AbdealiLoKo AbdealiLoKo enabled auto-merge (rebase) November 3, 2022 06:47
@AbdealiLoKo AbdealiLoKo merged commit 131d7d0 into corridor:master Nov 3, 2022
@15jgme
Copy link
Copy Markdown
Contributor Author

15jgme commented Nov 3, 2022

Thanks for the review!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants