Skip to content

Comments

MAINT: Set requires-pixi in pixi.toml#11154

Merged
jsignell merged 2 commits intopydata:mainfrom
VeckoTheGecko:requires-pixi
Feb 23, 2026
Merged

MAINT: Set requires-pixi in pixi.toml#11154
jsignell merged 2 commits intopydata:mainfrom
VeckoTheGecko:requires-pixi

Conversation

@VeckoTheGecko
Copy link
Contributor

@VeckoTheGecko VeckoTheGecko commented Feb 10, 2026

I recently found the requires-pixi field in the pixi.toml

We don't have to pin the pixi version in the workflow - compatability of Pixi versions can be enforced by Pixi itself.

cc @keewis

I think for robustness we'd probably want Parcels-code/pixi-lock#13 as well - let me know if that's a blocker for this PR. not needed

@github-actions github-actions bot added topic-documentation Automation Github bots, testing workflows, release automation labels Feb 10, 2026
@VeckoTheGecko VeckoTheGecko marked this pull request as draft February 10, 2026 12:32
We don't have to pin the pixi version in the workflow - compatability of
Pixi versions can be managed by Pixi itself.
@VeckoTheGecko VeckoTheGecko marked this pull request as ready for review February 12, 2026 10:32
Copy link
Member

@jsignell jsignell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine to me. Do we need a plan to keep the pixi version in the Dockerfile in sync with pixi.toml?

@VeckoTheGecko
Copy link
Contributor Author

pixi version in the Dockerfile in sync with pixi.toml?

Nope :) . I think I was the only one using the dockerfile anyway. If it gets out of sync, someone can raise an issue manually - the error message will be very clear anyway

(can you merge? I don't have access yet)

@jsignell jsignell merged commit d20fdb0 into pydata:main Feb 23, 2026
38 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Automation Github bots, testing workflows, release automation topic-documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants