Skip to content

Commit 17d7b08

Browse files
authored
Release threshold folder imports nit (#56400)
Cleans up release threshold model imports & adds related name to the relase_threshold table for projects
1 parent a95255e commit 17d7b08

File tree

9 files changed

+9
-11
lines changed

9 files changed

+9
-11
lines changed

fixtures/backup/model_dependencies/sorted.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,6 @@
167167
"sentry.PullRequest",
168168
"sentry.PullRequestCommit",
169169
"sentry.PullRequestComment",
170-
"sentry.ReleaseThreshold",
171170
"sentry.ReleaseActivity",
172171
"sentry.ReleaseCommit",
173172
"sentry.ReleaseEnvironment",
@@ -189,6 +188,7 @@
189188
"sentry.AlertRuleActivity",
190189
"sentry.TeamKeyTransaction",
191190
"sentry.MonitorEnvironment",
191+
"sentry.ReleaseThreshold",
192192
"sentry.MonitorCheckIn",
193193
"sentry.AlertRuleExcludedProjects",
194194
"sentry.Incident",

src/sentry/api/endpoints/release_threshold.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
ReleaseThresholdType,
1919
TriggerType,
2020
)
21-
from sentry.models.release_threshold.releasethreshold import ReleaseThreshold
21+
from sentry.models.release_threshold.release_threshold import ReleaseThreshold
2222

2323

2424
class ReleaseThresholdPOSTSerializer(serializers.Serializer):

src/sentry/api/endpoints/release_threshold_details.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
from sentry.api.exceptions import ResourceDoesNotExist
1212
from sentry.api.serializers import serialize
1313
from sentry.models.project import Project
14-
from sentry.models.release_threshold.releasethreshold import ReleaseThreshold
14+
from sentry.models.release_threshold.release_threshold import ReleaseThreshold
1515

1616

1717
@region_silo_endpoint

src/sentry/api/serializers/models/release_threshold.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
THRESHOLD_TYPE_INT_TO_STR,
44
TRIGGER_TYPE_INT_TO_STR,
55
)
6-
from sentry.models.release_threshold.releasethreshold import ReleaseThreshold
6+
from sentry.models.release_threshold.release_threshold import ReleaseThreshold
77

88

99
@register(ReleaseThreshold)

src/sentry/models/__init__.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,6 @@
9393
from .recentsearch import * # NOQA
9494
from .relay import * # NOQA
9595
from .release import * # NOQA
96-
from .release_threshold.constants import * # NOQA
97-
from .release_threshold.releasethreshold import * # NOQA
9896
from .releaseactivity import * # NOQA
9997
from .releasecommit import * # NOQA
10098
from .releaseenvironment import * # NOQA

src/sentry/models/release_threshold/constants.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ class TriggerType:
5252
ABSOLUTE_UNDER_STR = "absolute_under"
5353

5454
@classmethod
55-
def as_choices(cls):
55+
def as_choices(cls): # choices for model column
5656
return (
5757
(cls.PERCENT_OVER_STR, cls.PERCENT_OVER),
5858
(cls.PERCENT_UNDER_STR, cls.PERCENT_UNDER),
@@ -61,7 +61,7 @@ def as_choices(cls):
6161
)
6262

6363
@classmethod
64-
def as_str_choices(cls):
64+
def as_str_choices(cls): # choices for serializer
6565
return (
6666
(cls.PERCENT_OVER_STR, cls.PERCENT_OVER_STR),
6767
(cls.PERCENT_UNDER_STR, cls.PERCENT_UNDER_STR),
@@ -72,7 +72,7 @@ def as_str_choices(cls):
7272

7373
THRESHOLD_TYPE_INT_TO_STR = {
7474
ReleaseThresholdType.TOTAL_ERROR_COUNT: ReleaseThresholdType.TOTAL_ERROR_COUNT_STR,
75-
ReleaseThresholdType.NEW_ISSUE_COUNT_STR: ReleaseThresholdType.NEW_ISSUE_COUNT_STR,
75+
ReleaseThresholdType.NEW_ISSUE_COUNT: ReleaseThresholdType.NEW_ISSUE_COUNT_STR,
7676
ReleaseThresholdType.UNHANDLED_ISSUE_COUNT: ReleaseThresholdType.UNHANDLED_ISSUE_COUNT_STR,
7777
ReleaseThresholdType.REGRESSED_ISSUE_COUNT: ReleaseThresholdType.REGRESSED_ISSUE_COUNT_STR,
7878
ReleaseThresholdType.FAILURE_RATE: ReleaseThresholdType.FAILURE_RATE_STR,

tests/sentry/api/endpoints/test_release_threshold.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
from django.urls import reverse
22

33
from sentry.models.environment import Environment
4-
from sentry.models.release_threshold.releasethreshold import ReleaseThreshold
4+
from sentry.models.release_threshold.release_threshold import ReleaseThreshold
55
from sentry.testutils.cases import APITestCase
66

77

tests/sentry/api/endpoints/test_release_threshold_details.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
from django.urls import reverse
22

33
from sentry.models.environment import Environment
4-
from sentry.models.release_threshold.releasethreshold import ReleaseThreshold
4+
from sentry.models.release_threshold.release_threshold import ReleaseThreshold
55
from sentry.testutils.cases import APITestCase
66

77

0 commit comments

Comments
 (0)