elastic / apm-agent-dotnet

https://www.elastic.co/guide/en/apm/agent/dotnet/current/index.html
Apache License 2.0
573 stars 205 forks source link

Added check that Open Telemetry Bridge is enable and only then display warning #2355

Closed zviedris closed 1 month ago

zviedris commented 1 month ago

Added check so that warning is only displayed when configuration is enabled. Now warning is displayed in all cases and is misleading.

cla-checker-service[bot] commented 1 month ago

❌ Author of the following commits did not sign a Contributor Agreement: 03025769b32f35c5b7da3ea2006c7c2b9a2eb76c

Please, read and sign the above mentioned agreement if you want to contribute to this project

github-actions[bot] commented 1 month ago

👋 @zviedris Thanks a lot for your contribution!

It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it.

Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it.