-
Notifications
You must be signed in to change notification settings - Fork 0
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
dotenv integration #68
Comments
suggestion for the revlock problem is to change the deps here to >= instead of == and not increment it on every patch. That will give downstream users the flexibility to use later versions of whatever or delay upgrading depending on what happens here. my 2p |
I preferred to keep dependencies strict to avoid compatibility issues, and ensure replicability. |
For the dotenv support, you can use your shell capabilities as a workaround. source ./tests/otel.env
uv run dotenv -f ./tests/otel.env run -- opentelemetry-instrument pytest tests -m integration {{FLAGS}} I will add a flag to allow passing a dotenv file. |
I understand what you are aiming at, but doing so makes conflicts downstream. most notably the pytest version. This may limit adoption even if it achieves what you mention. I'm not arguing, just saying because I don't think I can use it due to this, even if others can match versions with you whenever a release occurs. |
Right now, I am not using pytest_otel, yet. I would like to be able to use a dotenv file to provide variables as opposed to launching pytest with dotenv.
Right now, I run tests like this, with
opentelemetry-instrument
, and I get spans for httpx calls, but not pytest. I would like to try this plugin../tests/otel.env
Note: I have the following dev dependencies, which makes it incompatible with this plugin, as the dependencies here are strict
==
I understand this is a combination of a feature request (dotenv support) and dependency leniency (something besides ==), so if you want this split up I can do that.
The text was updated successfully, but these errors were encountered: