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

support Python 2.7, 3.7-3.11 in setup.py #56

Merged
merged 2 commits into from
Nov 4, 2022
Merged

support Python 2.7, 3.7-3.11 in setup.py #56

merged 2 commits into from
Nov 4, 2022

Conversation

dholth
Copy link
Contributor

@dholth dholth commented Oct 24, 2022

Description

Why not declare support for new Pythons.

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Oct 24, 2022
include \
setup.h CHANGELOG CODE_OF_CONDUCT.md HOW_WE_USE_GITHUB.md LICENSE \
Makefile README.rst picosat.c picosat.h pycosat.c test_pycosat.py
graft examples/
Copy link
Contributor Author

@dholth dholth Oct 25, 2022

Choose a reason for hiding this comment

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

basically git ls-files

Otherwise it doesn't include picosat.c and picosat.h, necessary to compile the extension.

@dholth dholth requested a review from mcg1969 October 25, 2022 13:58
@dholth dholth added the in-progress issue is actively being worked on label Nov 4, 2022
"Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should we give up and just say Python :: 3 and requires-python > 3.7

Copy link

Choose a reason for hiding this comment

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

Nah I think this is fine

@dholth dholth merged commit 6867bdc into main Nov 4, 2022
@github-actions github-actions bot added the locked [bot] locked due to inactivity label Nov 5, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed [bot] added once the contributor has signed the CLA in-progress issue is actively being worked on locked [bot] locked due to inactivity
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants