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

Black update, updating Python versions, setuptools_scm recommendations #53

Merged
merged 10 commits into from May 17, 2022

Conversation

imrehg
Copy link
Member

@imrehg imrehg commented May 13, 2022

Tests outcome depends on facultyai/faculty#201 being released.

@imrehg
Copy link
Member Author

imrehg commented May 13, 2022

RuntimeError: support for python < 3.6 has been removed in setuptools_scm>=6.0.0

Should Python 2.7 and 3.5 be completely dropped from this library as well, then?

@imrehg imrehg changed the title Black update and adding newer Pythin versions. Black update and updating Python versions. May 13, 2022
@imrehg imrehg changed the title Black update and updating Python versions. Black update, updating Python versions, setuptools_scm recommendations May 13, 2022
@acroz
Copy link
Member

acroz commented May 13, 2022

LGTM, pending tests being fixed. You could also consider removing the six dependency and imports from __future__ since you're removing Python 2 support, but these are not urgent.

@imrehg
Copy link
Member Author

imrehg commented May 13, 2022

@acroz done the changes you mentioned, good shout! Let me know if anything else needs changing.

@acroz
Copy link
Member

acroz commented May 13, 2022

@imrehg Can probably update the faculty dependency to be 0.30.2 or later and rerun the tests.

I also realised that you can remove the universal wheel setting from setup.cfg as we're not supporting Python 2.

@imrehg
Copy link
Member Author

imrehg commented May 13, 2022

@acroz ah, good shout on that, removed, and bumped, all in the clear now.

@acroz acroz merged commit 143246b into facultyai:master May 17, 2022
@acroz
Copy link
Member

acroz commented May 17, 2022

Thanks, released as 0.14.1

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

Successfully merging this pull request may close these issues.

None yet

2 participants