Please pylint and pyright at the same time

Author: xrmx

opentelemetry-sdk/src/opentelemetry/sdk/trace/__init__.py

@@ -89,7 +89,8 @@
 _ENV_VALUE_UNSET = ""
 
 
-class SpanProcessor:
+@typing.runtime_checkable
+class SpanProcessor(typing.Protocol):
     """Interface which allows hooks for SDK's `Span` start and end method
     invocations.
 
@@ -112,7 +113,6 @@ def on_start(
             span: The :class:`opentelemetry.trace.Span` that just started.
             parent_context: The parent context of the span that just started.
         """
-        ...
 
     def on_end(self, span: "ReadableSpan") -> None:
         """Called when a :class:`opentelemetry.trace.Span` is ended.
@@ -123,11 +123,9 @@ def on_end(self, span: "ReadableSpan") -> None:
         Args:
             span: The :class:`opentelemetry.trace.Span` that just ended.
         """
-        ...
 
     def shutdown(self) -> None:
         """Called when a :class:`opentelemetry.sdk.trace.TracerProvider` is shutdown."""
-        ...
 
     def force_flush(self, timeout_millis: int = 30000) -> bool:
         """Export all ended spans to the configured Exporter that have not yet
@@ -140,7 +138,6 @@ def force_flush(self, timeout_millis: int = 30000) -> bool:
         Returns:
             False if the timeout is exceeded, True otherwise.
         """
-        ...
 
 
 # Temporary fix until https://github.com/PyCQA/pylint/issues/4098 is resolved

opentelemetry-sdk/src/opentelemetry/sdk/trace/export/__init__.py

@@ -51,7 +51,8 @@ class SpanExportResult(Enum):
     FAILURE = 1
 
 
-class SpanExporter:
+@typing.runtime_checkable
+class SpanExporter(typing.Protocol):
     """Interface for exporting spans.
 
     Interface to be implemented by services that want to export spans recorded
@@ -72,21 +73,18 @@ def export(
         Returns:
             The result of the export
         """
-        ...
 
     def shutdown(self) -> None:
         """Shuts down the exporter.
 
         Called when the SDK is shut down.
         """
-        ...
 
     def force_flush(self, timeout_millis: int = 30000) -> bool:
         """Hint to ensure that the export of any spans the exporter has received
         prior to the call to ForceFlush SHOULD be completed as soon as possible, preferably
         before returning from this method.
         """
-        ...
 
 
 class SimpleSpanProcessor(SpanProcessor):