Correct type hints per PEP 484 (#2748) (#2759)

* Correct type hints per PEP 484 ('https://www.python.org/dev/peps/pep-0484/#suggested-syntax-for-python-2-7-and-straddling-code')

* Adding release notes

* Code Review Suggestion

* 😳

Co-authored-by: Brett Langdon <[email protected]>
(cherry picked from commit 19f4c5c)

Co-authored-by: Chris Morris <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Author: mergify[bot]

ddtrace/contrib/trace_utils.py

@@ -306,7 +306,7 @@ def set_flattened_tags(
     items,  # type: Iterator[Tuple[str, Any]]
     sep=".",  # type: str
     exclude_policy=None,  # type: Optional[Callable[[str], bool]]
-    processor=None,  # type Optional[Callable[[Any], Any]]
+    processor=None,  # type: Optional[Callable[[Any], Any]]
 ):
     # type: (...) -> None
     for prefix, value in items:

ddtrace/internal/processor/__init__.py

@@ -15,7 +15,7 @@ class SpanProcessor(six.with_metaclass(abc.ABCMeta)):
     """A Processor is used to process spans as they are created and finished by a tracer."""
 
     def __attrs_post_init__(self):
-        # type () -> None
+        # type: () -> None
         """Default post initializer which logs the representation of the
         Processor at the ``logging.DEBUG`` level.
 

ddtrace/internal/processor/trace.py

@@ -21,7 +21,7 @@
 @attr.s
 class TraceProcessor(six.with_metaclass(abc.ABCMeta)):
     def __attrs_post_init__(self):
-        # type () -> None
+        # type: () -> None
         """Default post initializer which logs the representation of the
         TraceProcessor at the ``logging.DEBUG`` level.
 

ddtrace/runtime/__init__.py

@@ -8,7 +8,7 @@
 class _RuntimeMetricsStatus(type):
     @property
     def _enabled(_):
-        # type () -> bool
+        # type: () -> bool
         """Runtime metrics enabled status."""
         return ddtrace.internal.runtime.runtime_metrics.RuntimeWorker.enabled
 

ddtrace/tracer.py

@@ -234,12 +234,12 @@ async def web_handler(request):
         return ctx
 
     def current_trace_context(self, *args, **kwargs):
-        # type (...) -> Optional[Context]
+        # type: (...) -> Optional[Context]
         """Return the context for the current trace.
 
         If there is no active trace then None is returned.
         """
-        active = self.context_provider.active(*args, **kwargs)
+        active = self.context_provider.active()
         if isinstance(active, Context):
             return active
         elif isinstance(active, Span):

releasenotes/notes/correct-type-annotations-2ee30779966960b6.yaml

@@ -0,0 +1,4 @@
+---
+fixes:
+  - |
+    Corrected some typing annotations for PEP 484 compliance