-
Notifications
You must be signed in to change notification settings - Fork 186
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
feat: adds the ability to set the index strategy #1986
Conversation
…eat/index-strategy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good to me. Ony had minor comments
Co-authored-by: Hofer-Julian <[email protected]>
BTW I forgot, but before merging we need to change the schema and add docs. |
Unrelated to this PR, but I already wondered why we generate the schema from pydantic instead from our Rust struct. I assume that's not easy to do with serde? |
I think the last time we checked that crate was less expressive? |
Done. |
This PR enables to set the UV
index-strategy
from pixi. I would like a test for this so it is still in draft for now :)closes: #1377