Skip to content
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

fix(types): Replace custom TYPE_CHECKING with stdlib typing.TYPE_CHECKING #3447

Merged
merged 5 commits into from
Aug 27, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion scripts/init_serverless_sdk.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
import re

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.integrations.aws_lambda import AwsLambdaIntegration

if TYPE_CHECKING:
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/_compat.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import sys

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/_queue.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@
from collections import deque
from time import time

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
5 changes: 1 addition & 4 deletions sentry_sdk/_types.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,4 @@
try:
from typing import TYPE_CHECKING
except ImportError:
TYPE_CHECKING = False
from typing import TYPE_CHECKING


# Re-exported for compat, since code out there in the wild might use this variable.
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/_werkzeug.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
SUCH DAMAGE.
"""

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Dict
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/ai/monitoring.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from sentry_sdk import start_span
from sentry_sdk.tracing import Span
from sentry_sdk.utils import ContextVar
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Optional, Callable, Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/ai/utils.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
from sentry_sdk.crons import monitor


from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from collections.abc import Mapping
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/attachments.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import os
import mimetypes

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.envelope import Item, PayloadRef

if TYPE_CHECKING:
Expand Down
4 changes: 2 additions & 2 deletions sentry_sdk/client.py
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are two import statements in this file, so it's better to remove one of them, right?

Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
from sentry_sdk.monitor import Monitor
from sentry_sdk.spotlight import setup_spotlight

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down Expand Up @@ -881,7 +881,7 @@ def __exit__(self, exc_type, exc_value, tb):
self.close()


from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
# Make mypy, PyCharm and other static analyzers think `get_options` is a
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/consts.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import itertools

from enum import Enum
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

# up top to prevent circular import due to integration import
DEFAULT_MAX_VALUE_LENGTH = 1024
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/crons/api.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import uuid

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING


if TYPE_CHECKING:
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/crons/decorator.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
from functools import wraps
from inspect import iscoroutinefunction

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.crons import capture_checkin
from sentry_sdk.crons.consts import MonitorStatus
from sentry_sdk.utils import now
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/envelope.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
import json
import mimetypes

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.session import Session
from sentry_sdk.utils import json_dumps, capture_internal_exceptions

Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/hub.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
ContextVar,
)

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/__init__.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
from abc import ABC, abstractmethod
from threading import Lock

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.utils import logger


Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/_asgi_common.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

from sentry_sdk.scope import should_send_default_pii
from sentry_sdk.integrations._wsgi_common import _filter_headers
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/_wsgi_common.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
import sentry_sdk
from sentry_sdk.scope import should_send_default_pii
from sentry_sdk.utils import AnnotatedValue, logger
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

try:
from django.http.request import RawPostDataException
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/aiohttp.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
except ImportError:
raise DidNotEnable("AIOHTTP not installed")

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from aiohttp.web_request import Request
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/argv.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from sentry_sdk.integrations import Integration
from sentry_sdk.scope import add_global_event_processor

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Optional
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/ariadne.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
event_from_exception,
package_version,
)
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

try:
# importing like this is necessary due to name shadowing in ariadne
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/arq.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import sys

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.consts import OP, SPANSTATUS
from sentry_sdk.integrations import DidNotEnable, Integration
from sentry_sdk.integrations.logging import ignore_logger
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/asgi.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
from functools import partial

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.api import continue_trace
from sentry_sdk.consts import OP

Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/asyncio.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
import sentry_sdk
from sentry_sdk.consts import OP
from sentry_sdk.integrations import Integration, DidNotEnable
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.utils import event_from_exception, reraise

try:
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/atexit.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
from sentry_sdk.utils import logger
from sentry_sdk.integrations import Integration
from sentry_sdk.utils import ensure_integration_enabled
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/aws_lambda.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
)
from sentry_sdk.integrations import Integration
from sentry_sdk.integrations._wsgi_common import _filter_headers
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/beam.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
event_from_exception,
reraise,
)
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/boto3.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from sentry_sdk.integrations import Integration, DidNotEnable
from sentry_sdk.tracing import Span

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.utils import (
capture_internal_exceptions,
ensure_integration_enabled,
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/bottle.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
from sentry_sdk.integrations import Integration, DidNotEnable
from sentry_sdk.integrations.wsgi import SentryWsgiMiddleware
from sentry_sdk.integrations._wsgi_common import RequestExtractor
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from sentry_sdk.integrations.wsgi import _ScopedResponse
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/celery/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
from sentry_sdk.integrations.celery.utils import _now_seconds_since_epoch
from sentry_sdk.integrations.logging import ignore_logger
from sentry_sdk.tracing import BAGGAGE_HEADER_NAME, TRANSACTION_SOURCE_TASK
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.tracing_utils import Baggage
from sentry_sdk.utils import (
capture_internal_exceptions,
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/celery/beat.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
_get_humanized_interval,
_now_seconds_since_epoch,
)
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.utils import (
logger,
match_regex_list,
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/celery/utils.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import time
from typing import cast

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Any, Tuple
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/chalice.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
parse_version,
reraise,
)
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

try:
import chalice # type: ignore
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/clickhouse_driver.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
from sentry_sdk.consts import OP, SPANDATA
from sentry_sdk.integrations import Integration, DidNotEnable
from sentry_sdk.tracing import Span
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.scope import should_send_default_pii
from sentry_sdk.utils import capture_internal_exceptions, ensure_integration_enabled

Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/cloud_resource_context.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from sentry_sdk.api import set_context
from sentry_sdk.utils import logger

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Dict
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/cohere.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
from functools import wraps

from sentry_sdk import consts
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.ai.monitoring import record_token_usage
from sentry_sdk.consts import SPANDATA
from sentry_sdk.ai.utils import set_data_normalized
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/dedupe.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
from sentry_sdk.integrations import Integration
from sentry_sdk.scope import add_global_event_processor

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from typing import Optional
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/django/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from importlib import import_module

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.consts import OP, SPANDATA
from sentry_sdk.db.explain_plan.django import attach_explain_plan_to_span
from sentry_sdk.scope import add_global_event_processor, should_send_default_pii
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/django/asgi.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
from django.core.handlers.wsgi import WSGIRequest

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.consts import OP

from sentry_sdk.integrations.asgi import SentryAsgiMiddleware
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/django/middleware.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
from django import VERSION as DJANGO_VERSION

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.consts import OP
from sentry_sdk.utils import (
ContextVar,
Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/django/signals_handlers.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
from django.dispatch import Signal

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.consts import OP
from sentry_sdk.integrations.django import DJANGO_VERSION

Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/django/templates.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from django import VERSION as DJANGO_VERSION

import sentry_sdk
from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING
from sentry_sdk.consts import OP
from sentry_sdk.utils import ensure_integration_enabled

Expand Down
2 changes: 1 addition & 1 deletion sentry_sdk/integrations/django/transactions.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

import re

from sentry_sdk._types import TYPE_CHECKING
from typing import TYPE_CHECKING

if TYPE_CHECKING:
from django.urls.resolvers import URLResolver
Expand Down
Loading
Loading