From 68ca0420f707651a3d59f510c89039650fd1141e Mon Sep 17 00:00:00 2001 From: David Doty Date: Mon, 10 Aug 2026 10:42:35 -0500 Subject: [PATCH] fixes #326; build with `python -m build` instead of deprecated `python setup.py sdist bdist_wheel` Invoking setup.py directly is deprecated, so this job was on a clock: whenever setuptools removes the setup.py command interface, it starts failing on setuptools' schedule rather than ours. `python -m build` is the standard PEP 517 front end and produces the same sdist and wheel in an isolated build environment. It is also what release.yml, the workflow that actually publishes to PyPI, already uses; having the workflow meant to *check* packaging build differently from the one that performs it somewhat defeated the point of the check. Verified locally in a clean virtualenv: `python -m build` against the current setup.py produces scadnano-0.21.1.tar.gz and scadnano-0.21.1-py3-none-any.whl, with the wheel containing __init__.py, scadnano.py, modifications.py and origami_rectangle.py. Co-Authored-By: Claude Opus 5 (1M context) --- .github/workflows/check_pypi_packaging.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/check_pypi_packaging.yml b/.github/workflows/check_pypi_packaging.yml index 3031336..42f5cd4 100644 --- a/.github/workflows/check_pypi_packaging.yml +++ b/.github/workflows/check_pypi_packaging.yml @@ -16,6 +16,6 @@ jobs: - name: Install dependencies run: | python -m pip install --upgrade pip - pip install setuptools wheel + pip install build - name: Build - run: python setup.py sdist bdist_wheel + run: python -m build