Skip to content

Commit

Permalink
Require Python >= 3.9
Browse files Browse the repository at this point in the history
Since numpy does also require it now, and 3.8 EOL is already in October, this seems acceptable to me.
  • Loading branch information
skogler committed Aug 3, 2024
1 parent 260440f commit 940864d
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 9 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [windows-latest]
# os: [ubuntu-latest, windows-latest, macos-13, macos-14]
os: [ubuntu-latest, windows-latest, macos-13, macos-14]

steps:
- uses: actions/checkout@v4
Expand Down
6 changes: 0 additions & 6 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,6 @@ requires = [
build-backend = "setuptools.build_meta"

[tool.cibuildwheel]
# Cannot run tests on arm64 macosx python 3.8
# See https://github.com/pypa/cibuildwheel/pull/1169
test-skip=[
"cp38-macosx_*:arm64",
"cp*",
]
test-requires=[
"pytest"
]
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ def get_readme_contents():
long_description_content_type='text/markdown',
license='ISC',
ext_modules=ext_modules,
python_requires='>=3.8',
python_requires='>=3.9',
install_requires=['numpy'],
extras_require={'test': 'pytest'},
cmdclass=dict(build_ext=build_ext),
Expand Down

0 comments on commit 940864d

Please sign in to comment.