-
Notifications
You must be signed in to change notification settings - Fork 699
Handle None in OTEL_PROPAGATORS #4553
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
Conversation
@suauk Please add a changelog entry and fix the mypy warning |
@emdneto there's a couple failed checks, both seem to be unrelated to any changes in the PR, both seem to be related to invalid github links, I'm not too sure what to do about those, do you have any guidance? |
No idea. Probably a problem with the github itself |
Description
Fixes #4143
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
tox -e py312-test-opentelemetry-api
tox -e py312-test-opentelemetry-sdk
tox -e mypy
Does This PR Require a Contrib Repo Change?
Checklist: