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

fix: pypi dependencies not being removed #952

Merged

Conversation

tdejager
Copy link
Contributor

This fixes the situation where pypi dependencies were not removed from the prefix, while they were removed in the toml.

The problem was that if you have a single pypi dependency and removed this, the project would say it does not have pypi dependencies and the updating of the pypi prefix was never done.

This bug was introduced by me in #863, I think it would be nice to skip this code in the right situation, but seeing as this is a pretty big bug. I think its prudent to use this PR for now.
But hey 🤷‍♂️ , at least we have an add/remove tested now, and the test cli has expanded a bit.

@ruben-arts I've noticed there is also not a test for the conda situation here, maybe we should add it as well.

@tdejager tdejager changed the title Fixes pypi dependencies not being removed fix: pypi dependencies not being removed Mar 11, 2024
Copy link
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

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

Look good to me! Thanks for adding a test.

@ruben-arts ruben-arts merged commit 647496b into prefix-dev:main Mar 11, 2024
14 of 15 checks passed
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