Skip to content

Commit

Permalink
Merge branch 'master' into ryan953/update-react-aria
Browse files Browse the repository at this point in the history
  • Loading branch information
ryan953 authored Jul 30, 2024
2 parents 9b4fa2f + c6effcc commit 8416a43
Show file tree
Hide file tree
Showing 787 changed files with 14,143 additions and 7,552 deletions.
28 changes: 28 additions & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,17 @@ module.exports = {
'error',
{additionalHooks: '(useEffectAfterFirstRender|useMemoWithPrevious)'},
],
'no-restricted-imports': [
'error',
{
patterns: [
{
group: ['sentry/components/devtoolbar/*'],
message: 'Do not depend on toolbar internals',
},
],
},
],

// TODO(@anonrig): Remove this from eslint-sentry-config
'space-infix-ops': 'off',
Expand All @@ -44,6 +55,23 @@ module.exports = {
// and formatting these files.
ignorePatterns: ['*.json'],
overrides: [
{
files: ['static/app/components/devtoolbar/**/*.{ts,tsx}'],
rules: {
'no-restricted-imports': [
'error',
{
paths: [
{
name: 'sentry/utils/queryClient',
message:
'Import from `@tanstack/react-query` and `./hooks/useFetchApiData` or `./hooks/useFetchInfiniteApiData` instead.',
},
],
},
],
},
},
{
files: ['static/**/*.spec.{ts,js}', 'tests/js/**/*.{ts,js}'],
extends: ['plugin:testing-library/react', 'sentry-app/strict'],
Expand Down
7 changes: 3 additions & 4 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,7 @@
"python.testing.pytestEnabled": true,
"python.testing.unittestEnabled": false,
"python.testing.pytestPath": "${workspaceFolder}/.venv/bin/pytest",
"python.testing.pytestArgs": [
"tests"
],
"python.analysis.autoImportCompletions": true
"python.testing.pytestArgs": ["tests"],
"python.analysis.autoImportCompletions": true,
"prettier.configPath": "package.json"
}
3 changes: 0 additions & 3 deletions api-docs/openapi.json
Original file line number Diff line number Diff line change
Expand Up @@ -117,9 +117,6 @@
"/api/0/projects/{organization_id_or_slug}/{project_id_or_slug}/stats/": {
"$ref": "paths/projects/stats.json"
},
"/api/0/projects/{organization_id_or_slug}/{project_id_or_slug}/user-feedback/": {
"$ref": "paths/projects/user-feedback.json"
},
"/api/0/projects/{organization_id_or_slug}/{project_id_or_slug}/hooks/": {
"$ref": "paths/projects/service-hooks.json"
},
Expand Down
175 changes: 0 additions & 175 deletions api-docs/paths/projects/user-feedback.json

This file was deleted.

2 changes: 1 addition & 1 deletion fixtures/gitlab.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
from time import time

from sentry.integrations.gitlab.integration import GitlabIntegration
from sentry.integrations.models.integration import Integration
from sentry.models.identity import Identity, IdentityProvider
from sentry.models.integrations.integration import Integration
from sentry.models.repository import Repository
from sentry.silo.base import SiloMode
from sentry.testutils.cases import APITestCase
Expand Down
2 changes: 1 addition & 1 deletion fixtures/sdk_crash_detection/crash_event_dart.py
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ def get_crash_event_with_frames(frames: Sequence[Mapping[str, str]], **kwargs) -
"type": "os",
},
},
"sdk": {"name": "sentry.dart.flutter", "version": "8.2.0"},
"sdk": {"name": "sentry.dart.flutter", "version": "8.2.1"},
"timestamp": time.time(),
"type": "error",
}
Expand Down
2 changes: 1 addition & 1 deletion fixtures/vsts.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@
import pytest
import responses

from sentry.integrations.models.integration import Integration
from sentry.integrations.vsts import VstsIntegrationProvider
from sentry.integrations.vsts.integration import VstsIntegration
from sentry.models.integrations.integration import Integration
from sentry.silo.base import SiloMode
from sentry.testutils.cases import IntegrationTestCase
from sentry.testutils.helpers.integrations import get_installation_of_type
Expand Down
12 changes: 0 additions & 12 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,6 @@ module = [
"sentry.api.endpoints.group_integration_details",
"sentry.api.endpoints.group_integrations",
"sentry.api.endpoints.index",
"sentry.api.endpoints.integrations.install_request",
"sentry.api.endpoints.integrations.sentry_apps.details",
"sentry.api.endpoints.integrations.sentry_apps.index",
"sentry.api.endpoints.integrations.sentry_apps.installation.index",
Expand All @@ -143,8 +142,6 @@ module = [
"sentry.api.endpoints.integrations.sentry_apps.requests",
"sentry.api.endpoints.integrations.sentry_apps.stats.details",
"sentry.api.endpoints.internal.mail",
"sentry.api.endpoints.organization_code_mapping_codeowners",
"sentry.api.endpoints.organization_code_mappings",
"sentry.api.endpoints.organization_details",
"sentry.api.endpoints.organization_events",
"sentry.api.endpoints.organization_events_facets",
Expand All @@ -163,7 +160,6 @@ module = [
"sentry.api.endpoints.organization_projects",
"sentry.api.endpoints.organization_projects_experiment",
"sentry.api.endpoints.organization_releases",
"sentry.api.endpoints.organization_repositories",
"sentry.api.endpoints.organization_request_project_creation",
"sentry.api.endpoints.organization_search_details",
"sentry.api.endpoints.organization_sessions",
Expand Down Expand Up @@ -195,14 +191,10 @@ module = [
"sentry.api.permissions",
"sentry.api.serializers.models.auth_provider",
"sentry.api.serializers.models.dashboard",
"sentry.api.serializers.models.doc_integration",
"sentry.api.serializers.models.environment",
"sentry.api.serializers.models.event",
"sentry.api.serializers.models.external_actor",
"sentry.api.serializers.models.group",
"sentry.api.serializers.models.group_stream",
"sentry.api.serializers.models.integration",
"sentry.api.serializers.models.integration_feature",
"sentry.api.serializers.models.notification_action",
"sentry.api.serializers.models.organization",
"sentry.api.serializers.models.plugin",
Expand All @@ -213,7 +205,6 @@ module = [
"sentry.api.serializers.models.team",
"sentry.api.serializers.models.user",
"sentry.api.serializers.models.user_identity_config",
"sentry.api.serializers.rest_framework.doc_integration",
"sentry.api.serializers.rest_framework.mentions",
"sentry.api.serializers.rest_framework.notification_action",
"sentry.api.serializers.rest_framework.rule",
Expand Down Expand Up @@ -308,7 +299,6 @@ module = [
"sentry.integrations.pagerduty.integration",
"sentry.integrations.pipeline",
"sentry.integrations.slack.actions.form",
"sentry.integrations.slack.client",
"sentry.integrations.slack.integration",
"sentry.integrations.slack.message_builder.issues",
"sentry.integrations.slack.message_builder.notifications.digest",
Expand Down Expand Up @@ -338,7 +328,6 @@ module = [
"sentry.middleware.auth",
"sentry.middleware.ratelimit",
"sentry.middleware.superuser",
"sentry.models.integrations.external_issue",
"sentry.models.integrations.sentry_app_installation",
"sentry.models.organizationaccessrequest",
"sentry.monitors.consumers.monitor_consumer",
Expand Down Expand Up @@ -388,7 +377,6 @@ module = [
"sentry.sentry_apps.installations",
"sentry.sentry_metrics.consumers.indexer.slicing_router",
"sentry.sentry_metrics.indexer.postgres.postgres_v2",
"sentry.shared_integrations.client.base",
"sentry.shared_integrations.client.proxy",
"sentry.similarity.features",
"sentry.snuba.errors",
Expand Down
2 changes: 1 addition & 1 deletion requirements-base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ sentry-arroyo>=2.16.5
sentry-kafka-schemas>=0.1.101
sentry-ophio==0.2.7
sentry-redis-tools>=0.1.7
sentry-relay>=0.9.0
sentry-relay>=0.9.1
sentry-sdk>=2.11.0
slack-sdk>=3.27.2
snuba-sdk>=2.0.33
Expand Down
4 changes: 2 additions & 2 deletions requirements-dev-frozen.txt
Original file line number Diff line number Diff line change
Expand Up @@ -180,12 +180,12 @@ selenium==4.16.0
sentry-arroyo==2.16.5
sentry-cli==2.16.0
sentry-devenv==1.7.0
sentry-forked-django-stubs==5.0.2.post6
sentry-forked-django-stubs==5.0.2.post8
sentry-forked-djangorestframework-stubs==3.15.0.post1
sentry-kafka-schemas==0.1.101
sentry-ophio==0.2.7
sentry-redis-tools==0.1.7
sentry-relay==0.9.0
sentry-relay==0.9.1
sentry-sdk==2.11.0
sentry-usage-accountant==0.0.10
simplejson==3.17.6
Expand Down
2 changes: 1 addition & 1 deletion requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ pip-tools>=7.1.0
packaging>=21.3

# for type checking
sentry-forked-django-stubs>=5.0.2.post6
sentry-forked-django-stubs>=5.0.2.post8
sentry-forked-djangorestframework-stubs>=3.15.0.post1
lxml-stubs
msgpack-types>=0.2.0
Expand Down
2 changes: 1 addition & 1 deletion requirements-frozen.txt
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ sentry-arroyo==2.16.5
sentry-kafka-schemas==0.1.101
sentry-ophio==0.2.7
sentry-redis-tools==0.1.7
sentry-relay==0.9.0
sentry-relay==0.9.1
sentry-sdk==2.11.0
sentry-usage-accountant==0.0.10
simplejson==3.17.6
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/api/authentication.py
Original file line number Diff line number Diff line change
Expand Up @@ -404,7 +404,7 @@ def accepts_auth(self, auth: list[bytes]) -> bool:
return not token_str.startswith(SENTRY_ORG_AUTH_TOKEN_PREFIX)

def authenticate_token(self, request: Request, token_str: str) -> tuple[Any, Any]:
user: AnonymousUser | RpcUser | None = AnonymousUser()
user: AnonymousUser | User | RpcUser | None = AnonymousUser()

token: SystemToken | ApiTokenReplica | ApiToken | None = SystemToken.from_request(
request, token_str
Expand Down
Loading

0 comments on commit 8416a43

Please sign in to comment.