-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add repository.url
and repository.provider
labels
#97
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but can you also update the changelog?
# # because the library was already initialized with the OpenTelemetry tracker | ||
# set_tracker(TrackerType.OPENTELEMETRY) | ||
@pytest.mark.asyncio | ||
async def test_concurrency_tracking_opentelemetry(monkeypatch): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!! forgot we could uncomment this test after the otel update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah i went to update the data there and noticed we can use those now 🎉
Co-authored-by: Brett Beutell <[email protected]>
Co-authored-by: Brett Beutell <[email protected]>
Resolves #96