From b779d08c8c782ef178f4ff04e6d8e447f92dab65 Mon Sep 17 00:00:00 2001 From: Ryan Ly Date: Thu, 19 Dec 2024 15:45:35 -0800 Subject: [PATCH] Fix hdmf-zarr workflow (#1222) * Fix hdmf-zarr workflow * Update CHANGELOG.md --- .github/workflows/run_hdmf_zarr_tests.yml | 9 ++++----- CHANGELOG.md | 1 + 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/run_hdmf_zarr_tests.yml b/.github/workflows/run_hdmf_zarr_tests.yml index 5e76711af..6eb5546ab 100644 --- a/.github/workflows/run_hdmf_zarr_tests.yml +++ b/.github/workflows/run_hdmf_zarr_tests.yml @@ -21,7 +21,7 @@ jobs: - name: Set up Python uses: actions/setup-python@v5 with: - python-version: '3.10' # use 3.10 until hdmf-zarr updates versioneer.py which breaks on newer python + python-version: '3.13' - name: Update pip run: python -m pip install --upgrade pip @@ -29,10 +29,9 @@ jobs: - name: Clone HDMF-Zarr and install dev branch of HDMF run: | python -m pip list - git clone https://github.com/hdmf-dev/hdmf-zarr.git --recurse-submodules + git clone https://github.com/hdmf-dev/hdmf-zarr.git cd hdmf-zarr - python -m pip install -r requirements-dev.txt # do not install the pinned install requirements - python -m pip install . # this will install a different version of hdmf from the current one + python -m pip install .[test] # this will install a different version of hdmf from the current one cd .. python -m pip uninstall -y hdmf # uninstall the other version of hdmf python -m pip install . # reinstall current branch of hdmf @@ -41,4 +40,4 @@ jobs: - name: Run HDMF-Zarr tests on HDMF-Zarr dev branch run: | cd hdmf-zarr - pytest + pytest -v diff --git a/CHANGELOG.md b/CHANGELOG.md index 438a0c7df..5c965f9dc 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,7 @@ - Added support for expandable datasets of references for untyped and compound data types. @stephprince [#1188](https://github.com/hdmf-dev/hdmf/pull/1188) - Improved html representation of data in `Container` objects. @h-mayorquin [#1100](https://github.com/hdmf-dev/hdmf/pull/1100) - Added error when using colon for `Container` name. A colon cannot be used as a group name when writing to Zarr on Windows. @stephprince [#1202](https://github.com/hdmf-dev/hdmf/pull/1202) +- Adjusted testing for hdmf-zarr. @rly [#1222](https://github.com/hdmf-dev/hdmf/pull/1222) ### Bug fixes - Fixed inaccurate error message when validating reference data types. @stephprince [#1199](https://github.com/hdmf-dev/hdmf/pull/1199)