TracerProvider(ABC) doc string correction#4894
TracerProvider(ABC) doc string correction#489442Questions wants to merge 4 commits intoopen-telemetry:mainfrom
Conversation
|
|
|
Isn't every instrumentation using |
Yes. I've purposefully done a minimal change, but as mentioned here, a less prescriptive docstring could be warranted:
That said, the current docstring is incorrect: @xrmx Would it make sense to still include a warning about using |
|
@xrmx Can the changelog be skipped for this? |
|
@42Questions I've opened open-telemetry/opentelemetry-python-contrib#4173 to pass an explicit value instead of |
Description
Correction to
class TracerProvider(ABC)docstring, extensively discussed but never implemented open-telemetry/opentelemetry.io#2517 .Correction to
class TracerProvider(ABC).get_meterdocstring as well with the same mistake.