-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Vendor new pluggy version #1637
Comments
This needs a new release of pluggy first, which @hpk42 and me got distracted from doing at the end of the sprint. |
@nicoddemus |
i mailed @hpk42 about moving it to pytest-dev and adding more if pytest-core to the pypi maintainers |
As @hpk42 is on vacation, I'm moving this to 3.1 as I don't think it should block the 3.0 release. |
Couldn't we just vendor the current git state FWIW? |
Not sure it's a good idea... currently we show pluggy's version on pytest's output, so it would be weird to display a hash at that point... |
Fair enough - let's postpone it to 3.1 then. |
From the ML:
|
Nice I've been meaning to suggest reversed import logic in |
I think the point of vendoring is to isolate different pluggy installations in the same environment. For example, I right now both pytest and tox use pluggy in some vendored version, so different (and incompatible) versions of pluggy can coexist in the same virtual environment without problems. Hope that's clear, gotta go sleep in... like two hours ago already. 😬 |
@nicoddemus ahh gotcha. That use case didn't even cross my mind. I guess it seems odd to me because when would you ever not have your local vendored version? It's not like it makes any difference right now if you decide to install a dev version of Further, maybe the more correct solution is to get pluggy working across both packages? |
@tgoodlet bascially the exact pluggy api is supposed to be a implementation detail of tox/pytest that does not leak however the python module system makes such an abstraction impossible, thus we separate with way more crude mechanisms |
That's the plan for pluggy-1.0. Meanwhile the plan is to vendor it in pytest and tox, at least. |
Finally! 🎆 |
😁 |
As suggested in #704
The text was updated successfully, but these errors were encountered: