From 2e8fb6bf9ac12ed60111e342cc11d2a83eca8243 Mon Sep 17 00:00:00 2001 From: Max Ghenis Date: Sat, 24 Jan 2026 18:17:32 -0500 Subject: [PATCH] Remove stale smoke test workaround - Remove Python 3.13 special handling (tables now has 3.13 wheels) - Remove unused dependencies: synthimpute, click, pathlib, pytest-dependency, tabulate - Remove stale comment in setup.py Co-Authored-By: Claude Opus 4.5 --- .github/workflows/pr.yaml | 13 +------------ changelog_entry.yaml | 4 ++++ setup.py | 2 -- 3 files changed, 5 insertions(+), 14 deletions(-) diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml index 92f9fb735..7166e1ca8 100644 --- a/.github/workflows/pr.yaml +++ b/.github/workflows/pr.yaml @@ -82,18 +82,7 @@ jobs: - name: Verify pytest plugins run: python -m pytest --version - name: Install -us package from PyPI - run: | - if [[ "${{ matrix.python-version }}" == "3.13" ]]; then - # For Python 3.13, install newer tables first and ignore conflicts - pip install "tables>=3.10.1" - pip install policyengine-us --no-deps - # Install remaining dependencies manually - pip install click==8.1.3 pathlib pytest-dependency synthimpute tabulate - pip install policyengine-us-data --no-deps - else - python -m pip install policyengine-us - fi - shell: bash + run: python -m pip install policyengine-us - name: Run smoke tests only run: python -m pytest -m smoke --reruns 2 --reruns-delay 5 -v -s env: diff --git a/changelog_entry.yaml b/changelog_entry.yaml index e69de29bb..40a2fea83 100644 --- a/changelog_entry.yaml +++ b/changelog_entry.yaml @@ -0,0 +1,4 @@ +- bump: patch + changes: + removed: + - Stale smoke test workaround with unused dependencies (synthimpute, etc.). diff --git a/setup.py b/setup.py index 31f4b569c..f3d37c3b8 100644 --- a/setup.py +++ b/setup.py @@ -82,8 +82,6 @@ python_requires=">=3.10", extras_require={ "dev": dev_requirements, - # Note: For Python 3.13, policyengine-us requires special installation - # due to tables==3.9.2 not having Python 3.13 wheels. See CI workflow for workaround. }, include_package_data=True, # Will read MANIFEST.in install_requires=general_requirements,