Skip to content
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

Specify hatch envs using GitHub actions matrix for v3 tests #1728

Merged
merged 5 commits into from
Apr 8, 2024

Conversation

maxrjones
Copy link
Member

@maxrjones maxrjones commented Mar 27, 2024

Addresses @jhamman's request from #1656 (comment). Not the cleanest solution, but seems to be the only option until pypa/hatch#858 is resolved.

TODO:

  • Add unit tests and/or doctests in docstrings
  • Add docstrings and API docs for any new/modified user-facing classes and functions
  • New/modified features documented in docs/tutorial.rst
  • Changes documented in docs/release.rst
  • GitHub Actions have all passed
  • Test coverage is 100% (Codecov passes)

@MSanKeys963 MSanKeys963 added the V3 Related to compatibility with V3 spec label Mar 28, 2024
Co-authored-by: Joe Hamman <jhamman1@gmail.com>
@jhamman
Copy link
Member

jhamman commented Apr 7, 2024

Eeek, I may have been wrong:

Invalid workflow file: .github/workflows/test-v3.yml#L13
The workflow is not valid. .github/workflows/test-v3.yml (Line: 13, Col: 3): Unrecognized named-value: 'matrix'. Located at position 1 within expression: matrix.python-version

maxrjones and others added 3 commits April 7, 2024 09:10
@jhamman
Copy link
Member

jhamman commented Apr 8, 2024

Thanks @maxrjones!

@jhamman jhamman merged commit 4e6cca2 into zarr-developers:v3 Apr 8, 2024
13 of 14 checks passed
@maxrjones maxrjones deleted the v3-hatch-actions branch April 8, 2024 17:06
d-v-b pushed a commit to d-v-b/zarr-python that referenced this pull request Apr 10, 2024
…elopers#1728)

* Specify v3 hatch envs using GitHub actions matrix

* Update .github/workflows/test-v3.yml

Co-authored-by: Joe Hamman <jhamman1@gmail.com>

* Update .github/workflows/test-v3.yml

Co-authored-by: Joe Hamman <jhamman1@gmail.com>

* test on 3.12 too

* no 3.12

---------

Co-authored-by: Joe Hamman <jhamman1@gmail.com>
Co-authored-by: Joe Hamman <joe@earthmover.io>
@maxrjones maxrjones mentioned this pull request Apr 18, 2024
6 tasks
@jhamman jhamman added this to the 3.0.0.alpha milestone Apr 22, 2024
d-v-b added a commit that referenced this pull request Apr 22, 2024
* chore: add deprecation warnings to v3 classes / functions

* Resolve Mypy erorrs in `v3` branch (#1692)

* refactor(v3): Using appropriate types

* fix(v3): Typing fixes + minor code fixes

* fix(v3): _sync_iter works with coroutines

* docs(v3/store/core.py): clearer comment

* fix(metadata.py): Use Any outside TYPE_CHECKING for Pydantic

* fix(zarr/v3): correct zarr format + remove unused method

* fix(v3/store/core.py): Potential suggestion on handling str store_like

* refactor(zarr/v3): Add more typing

* ci(.pre-commit-config.yaml): zarr v3 mypy checks turned on in pre-commit

* Specify hatch envs using GitHub actions matrix for v3 tests (#1728)

* Specify v3 hatch envs using GitHub actions matrix

* Update .github/workflows/test-v3.yml

Co-authored-by: Joe Hamman <jhamman1@gmail.com>

* Update .github/workflows/test-v3.yml

Co-authored-by: Joe Hamman <jhamman1@gmail.com>

* test on 3.12 too

* no 3.12

---------

Co-authored-by: Joe Hamman <jhamman1@gmail.com>
Co-authored-by: Joe Hamman <joe@earthmover.io>

* black -> ruff format + cleanup (#1639)

* black -> ruff + cleanup

* format

* Preserve git blame

* pre-commit fix

* Remove outdated dev install docs from installation.rst and link to contributing.rst (#1643)

Co-authored-by: Joe Hamman <joe@earthmover.io>

* chore: remove old v3 implementation

* chore: remove more version-conditional logic

* chore: remove v3_storage_transformers.py again

---------

Co-authored-by: Daniel Jahn (dahn) <dahnjahn@gmail.com>
Co-authored-by: Max Jones <14077947+maxrjones@users.noreply.github.com>
Co-authored-by: Joe Hamman <jhamman1@gmail.com>
Co-authored-by: Joe Hamman <joe@earthmover.io>
Co-authored-by: Saransh Chopra <saransh0701@gmail.com>
Co-authored-by: Alden Keefe Sampson <aldenkeefesampson@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V3 Related to compatibility with V3 spec
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants