-
Notifications
You must be signed in to change notification settings - Fork 8
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
update to cython 3 #390
update to cython 3 #390
Changes from 3 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,8 +8,7 @@ jobs: | |
fail-fast: false | ||
matrix: | ||
os: [windows-latest, macos-latest] | ||
python-version: [3.7, 3.8, 3.9, "3.10", "3.11", "3.12"] | ||
python-arch: [x64] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I bet this is a holdover from when we were still doing x86 builds. Makes sense to remove! |
||
python-version: [3.8, 3.9, "3.10", "3.11", "3.12"] | ||
|
||
steps: | ||
- uses: actions/checkout@v4 | ||
|
@@ -20,11 +19,10 @@ jobs: | |
- name: Fetch git tags | ||
run: git fetch origin +refs/tags/*:refs/tags/* | ||
|
||
- name: Set up python ${{ matrix.python-version }} ${{ matrix.python-arch }} | ||
- name: Set up python ${{ matrix.python-version }} | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
architecture: ${{ matrix.python-arch }} | ||
|
||
- name: Install dependencies | ||
run: python -m pip install build | ||
|
@@ -34,7 +32,7 @@ jobs: | |
|
||
- uses: actions/upload-artifact@v1 | ||
with: | ||
name: Wheel for ${{ matrix.os }} ${{ matrix.python-version }} ${{ matrix.python-arch }} | ||
name: Wheel for ${{ matrix.os }} ${{ matrix.python-version }} | ||
path: dist | ||
|
||
build-sdist: | ||
|
@@ -52,7 +50,7 @@ jobs: | |
- name: Set up python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: 3.7 | ||
python-version: 3.12 | ||
|
||
- name: Install dependencies | ||
run: python -m pip install build | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,6 +10,7 @@ Unreleased Changes | |
called with an invalid resampling algorithm. We now fall back to the | ||
underlying GDAL functions' error messages. | ||
https://github.com/natcap/pygeoprocessing/issues/387 | ||
* Updated to Cython 3. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If we are removing testing/wheel-building support for python 3.7, should that also be noted here in HISTORY and also removed from the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ooh you're right |
||
|
||
2.4.3 (2024-03-06) | ||
------------------ | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
# coding=UTF-8 | ||
# cython: language_level=3 | ||
import logging | ||
import os | ||
import shutil | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python 3.7 is end-of-life, we already removed it from the test workflow. And
setup-python
couldn't find an ARM64 python 3.7 distribution.