Skip to content

Use pixi to simplify ci jobs#825

Closed
maxrjones wants to merge 8 commits intozarr-developers:mainfrom
maxrjones:more-pixi
Closed

Use pixi to simplify ci jobs#825
maxrjones wants to merge 8 commits intozarr-developers:mainfrom
maxrjones:more-pixi

Conversation

@maxrjones
Copy link
Member

Uses pixi in the non-zarr CI jobs as well, to simplify the setup

TODO:

  • Unit tests and/or doctests in docstrings
  • Tests pass locally
  • Docstrings and API docs for any new/modified user-facing classes and functions
  • Changes documented in docs/release.rst
  • Docs build locally
  • GitHub Actions CI passes
  • Test coverage to 100% (Codecov passes)

@maxrjones
Copy link
Member Author

I need to do more work on this PR before it's ready, switching it to draft now

@maxrjones maxrjones marked this pull request as draft March 17, 2026 21:16
@d-v-b
Copy link
Contributor

d-v-b commented Mar 17, 2026

sounds good! I trust you to merge this when everything is green

@codecov
Copy link

codecov bot commented Mar 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.33%. Comparing base (eab1742) to head (857feb8).

❌ Your project status has failed because the head coverage (99.33%) is below the target coverage (100.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #825   +/-   ##
=======================================
  Coverage   99.33%   99.33%           
=======================================
  Files          26       26           
  Lines         898      898           
=======================================
  Hits          892      892           
  Misses          6        6           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@maxrjones
Copy link
Member Author

This turned out to be hard for me to fix, so I decided a complete modernization was warranted. So this is superseded by #826, which removes the need for pixi usage.

@maxrjones maxrjones closed this Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants