-
Notifications
You must be signed in to change notification settings - Fork 16
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
Stop pinning core dependencies #408
Comments
Thanks for your report! I started looking into this. I know that we are relying on some not-necessarily-public peculiarities in setuptools and/or importlib-metadata, so we'll need to run some tests before committing to loosening these versions. |
would you recommend having fawltydeps installed in a different venv or with pipx (like poetry does)? Because then this would not be a problem any more. |
Any method should work, and we really do want to support installing FawltyDeps in the same venv as your project (when the versions of dependencies otherwise align). If you are considering creating a different venv just for FawltyDeps' sake, then I would say that you could just as well use pipx (AFAIK that is ~precisely what pipx actually does, in addition to modifying your Which method should be preferred really depends on what kind of projects you're working with, and how you prefer to work with them: If your project provides a dedicated developer environment and specifies its own dev dependencies, then we hope you will consider including FawltyDeps among these. Otherwise, having FawltyDeps generally available on your machine is certainly not a bad idea, and makes it easy to casually run FawltyDeps on any Python project. BTW, my local tests of your proposed change is successful, and PR #409 should fix this issue. The first commit in that PR just replicates your suggested changes above, so if you want, I'm happy to update the authorship of that commit to your name/email. |
Is your feature request related to a problem? Please describe.
When I install falwtydeps on my project, it uninstalls setuptools and import-libmetadata in order to install older version
Describe the solution you'd like
In theory, falwltydeps should work on newer version of its dependencies, so there's no reason to pin these to major versions.
We should replace
^
with>=
in pyproject.tomlDescribe alternatives you've considered
I could install falwltydeps in a different virtual environment, or use pipx
The text was updated successfully, but these errors were encountered: