Skip to content

Conversation

@amoutaux
Copy link

This prevents devpi-import from working with constrained indices.
See devpi/devpi#1064.

@amoutaux
Copy link
Author

Hello @fschulze,

Any chance a new version can be deployed with this change ?

Many thanks.

@fschulze
Copy link
Collaborator

A test and changelog entry would be nice to have first. Adding an export/import test might not be trivial though if one hasn't done it before. I hope to take a look later this week.

This prevents devpi-import from working with constrained
indices.
See devpi/devpi#1064.
@amoutaux
Copy link
Author

I updated the changelog. I also tested that devpi-import works fine with a .dataindex.json containing constrained indices after this commit, though I must admit I'm not capable of writing such test for now.

@amoutaux
Copy link
Author

Hello @fschulze, just a small poking so as not to get forgotten please 🙏. Happy holidays :)

@fschulze fschulze merged commit 664c4a8 into devpi:main Dec 27, 2024
@fschulze
Copy link
Collaborator

Merged and added a test.

@amoutaux
Copy link
Author

amoutaux commented Jan 6, 2025

Thank you very much ! I shall be waiting for the 2.0.2 release then :)

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