From c7d6ac3e2b70f8b48aa75237502f44dd5709e21e Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 10:02:56 -0800 Subject: [PATCH 01/10] fix quoting --- .github/workflows/pypi-build-artifacts.yml | 2 +- .github/workflows/svn-build-artifacts.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index c33b8041dc..a0a6d43ae2 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -70,7 +70,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3 sqlalchemy>=2.0.18,<3" + CIBW_TEST_REQUIRES: 'pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3 "sqlalchemy>=2.0.18,<3"' CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index 6a81cd7d3c..84da6c28ce 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -65,7 +65,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3 sqlalchemy>=2.0.18,<3" + CIBW_TEST_REQUIRES: 'pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3 "sqlalchemy>=2.0.18,<3"' CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain From e9fd5c43c390ef3398800033b6ae48eec86a5e18 Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 10:09:37 -0800 Subject: [PATCH 02/10] do we need sqlalchemy? --- .github/workflows/pypi-build-artifacts.yml | 2 +- .github/workflows/svn-build-artifacts.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index a0a6d43ae2..d59dc77ec8 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -70,7 +70,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_TEST_REQUIRES: 'pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3 "sqlalchemy>=2.0.18,<3"' + CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3" CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index 84da6c28ce..8a46f8cf6b 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -65,7 +65,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_TEST_REQUIRES: 'pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3 "sqlalchemy>=2.0.18,<3"' + CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3" CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain From 01f5690c26ab42cd6cfaad05cdb7f4ce6ee065f7 Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 10:19:30 -0800 Subject: [PATCH 03/10] lets try uv --- .github/workflows/pypi-build-artifacts.yml | 3 ++- .github/workflows/svn-build-artifacts.yml | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index d59dc77ec8..c88c6b5e3a 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -70,7 +70,8 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3" + # Install test dependencies from pyproject.toml dependency-groups + CIBW_BEFORE_TEST: "uv pip install --group dev" CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index 8a46f8cf6b..1c67265750 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -65,7 +65,8 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1 pytest-lazy-fixture==0.6.3" + # Install test dependencies from pyproject.toml dependency-groups + CIBW_BEFORE_TEST: "uv pip install --group dev" CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain From 070e536dba5c089032ac52d97a43ba6df1dac9af Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 10:24:41 -0800 Subject: [PATCH 04/10] include sqlalchemy in dev dep --- pyproject.toml | 1 + 1 file changed, 1 insertion(+) diff --git a/pyproject.toml b/pyproject.toml index 9d602df554..14cd882b34 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -118,6 +118,7 @@ dev = [ "mypy-boto3-glue>=1.28.18", "mypy-boto3-dynamodb>=1.28.18", "pyarrow-stubs>=20.0.0.20251107", # Remove when pyarrow >= 23.0.0 https://github.com/apache/arrow/pull/47609 + "sqlalchemy>=2.0.18,<3", ] # for mkdocs docs = [ From 2d44374fa2d9372c8b47532cebf06cca4e19f87e Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 11:22:05 -0800 Subject: [PATCH 05/10] update uv lock --- uv.lock | 2 ++ 1 file changed, 2 insertions(+) diff --git a/uv.lock b/uv.lock index 4b36e971aa..1d6605595b 100644 --- a/uv.lock +++ b/uv.lock @@ -3796,6 +3796,7 @@ dev = [ { name = "pytest-lazy-fixture" }, { name = "pytest-mock" }, { name = "requests-mock" }, + { name = "sqlalchemy" }, { name = "typing-extensions" }, ] docs = [ @@ -3880,6 +3881,7 @@ dev = [ { name = "pytest-lazy-fixture", specifier = "==0.6.3" }, { name = "pytest-mock", specifier = "==3.15.1" }, { name = "requests-mock", specifier = "==1.12.1" }, + { name = "sqlalchemy", specifier = ">=2.0.18,<3" }, { name = "typing-extensions", specifier = "==4.15.0" }, ] docs = [ From 42f72a0a67a4c8ff2f9ac1fbf79adfb9c5adb767 Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 11:22:11 -0800 Subject: [PATCH 06/10] try uv sync --- .github/workflows/pypi-build-artifacts.yml | 3 +-- .github/workflows/svn-build-artifacts.yml | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index c88c6b5e3a..b5a6c1725b 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -70,8 +70,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - # Install test dependencies from pyproject.toml dependency-groups - CIBW_BEFORE_TEST: "uv pip install --group dev" + CIBW_BEFORE_TEST: "uv sync --group dev --no-install-project" CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index 1c67265750..02e57393e2 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -65,8 +65,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - # Install test dependencies from pyproject.toml dependency-groups - CIBW_BEFORE_TEST: "uv pip install --group dev" + CIBW_BEFORE_TEST: "uv sync --group dev --no-install-project" CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain From b33abd6a757b2952f828848e598fd8e7b5d22ce2 Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 11:29:14 -0800 Subject: [PATCH 07/10] uv run pytest --- .github/workflows/pypi-build-artifacts.yml | 2 +- .github/workflows/svn-build-artifacts.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index b5a6c1725b..a20dc2a8d5 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -71,7 +71,7 @@ jobs: CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" CIBW_BEFORE_TEST: "uv sync --group dev --no-install-project" - CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" + CIBW_TEST_COMMAND: "uv run pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain CIBW_TEST_SKIP: "pp*" diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index 02e57393e2..9d9224974b 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -66,7 +66,7 @@ jobs: CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" CIBW_BEFORE_TEST: "uv sync --group dev --no-install-project" - CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py" + CIBW_TEST_COMMAND: "uv run pytest {project}/tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain CIBW_TEST_SKIP: "pp*" From f1409e11a1e505710f1c9a8d87fe8c6a440c8847 Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 11:35:36 -0800 Subject: [PATCH 08/10] relative path --- .github/workflows/pypi-build-artifacts.yml | 4 ++-- .github/workflows/svn-build-artifacts.yml | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index a20dc2a8d5..088ae09f9e 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -70,8 +70,8 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_BEFORE_TEST: "uv sync --group dev --no-install-project" - CIBW_TEST_COMMAND: "uv run pytest {project}/tests/avro/test_decoder.py" + CIBW_BEFORE_TEST: "uv sync --directory {project} --only-group dev --no-install-project" + CIBW_TEST_COMMAND: "uv run --directory {project} pytest tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain CIBW_TEST_SKIP: "pp*" diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index 9d9224974b..56d8f1af6b 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -65,8 +65,8 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_BEFORE_TEST: "uv sync --group dev --no-install-project" - CIBW_TEST_COMMAND: "uv run pytest {project}/tests/avro/test_decoder.py" + CIBW_BEFORE_TEST: "uv sync --directory {project} --only-group dev --no-install-project" + CIBW_TEST_COMMAND: "uv run --directory {project} pytest tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain CIBW_TEST_SKIP: "pp*" From 2be89278a0335e7c1820ffa80c0cd841f9f3422c Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 13:58:03 -0800 Subject: [PATCH 09/10] --all-extras --- .github/workflows/pypi-build-artifacts.yml | 2 +- .github/workflows/svn-build-artifacts.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index 088ae09f9e..d2948578ef 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -70,7 +70,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_BEFORE_TEST: "uv sync --directory {project} --only-group dev --no-install-project" + CIBW_BEFORE_TEST: "uv sync --directory {project} --all-extras --no-install-project" CIBW_TEST_COMMAND: "uv run --directory {project} pytest tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index 56d8f1af6b..cbb4ee2db1 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -65,7 +65,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_BEFORE_TEST: "uv sync --directory {project} --only-group dev --no-install-project" + CIBW_BEFORE_TEST: "uv sync --directory {project} --all-extras --no-install-project" CIBW_TEST_COMMAND: "uv run --directory {project} pytest tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain From be10d20629777beba17e642353617e341770c42d Mon Sep 17 00:00:00 2001 From: Kevin Liu Date: Wed, 14 Jan 2026 14:03:37 -0800 Subject: [PATCH 10/10] Revert "--all-extras" This reverts commit 2be89278a0335e7c1820ffa80c0cd841f9f3422c. --- .github/workflows/pypi-build-artifacts.yml | 2 +- .github/workflows/svn-build-artifacts.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pypi-build-artifacts.yml b/.github/workflows/pypi-build-artifacts.yml index d2948578ef..088ae09f9e 100644 --- a/.github/workflows/pypi-build-artifacts.yml +++ b/.github/workflows/pypi-build-artifacts.yml @@ -70,7 +70,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_BEFORE_TEST: "uv sync --directory {project} --all-extras --no-install-project" + CIBW_BEFORE_TEST: "uv sync --directory {project} --only-group dev --no-install-project" CIBW_TEST_COMMAND: "uv run --directory {project} pytest tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain diff --git a/.github/workflows/svn-build-artifacts.yml b/.github/workflows/svn-build-artifacts.yml index cbb4ee2db1..56d8f1af6b 100644 --- a/.github/workflows/svn-build-artifacts.yml +++ b/.github/workflows/svn-build-artifacts.yml @@ -65,7 +65,7 @@ jobs: # Ignore 32 bit architectures CIBW_ARCHS: "auto64" CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14" - CIBW_BEFORE_TEST: "uv sync --directory {project} --all-extras --no-install-project" + CIBW_BEFORE_TEST: "uv sync --directory {project} --only-group dev --no-install-project" CIBW_TEST_COMMAND: "uv run --directory {project} pytest tests/avro/test_decoder.py" # Ignore tests for pypy since not all dependencies are compiled for it # and would require a local rust build chain