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

chore(uptime): Move UptimeDomainCheckFailure to uptime/grouptype.py take 2 #88821

Merged
merged 4 commits into from
Apr 4, 2025

Conversation

wedamija
Copy link
Member

@wedamija wedamija commented Apr 4, 2025

We want to keep grouptypes in the module that is related to them, so just moving this out.

This caused pickling problems last time, so importing the model in its old location to prevent issues.

…ake 2

We want to keep grouptypes in the module that is related to them, so just moving this out.

This caused pickling problems last time, so importing the model in its old location to prevent issues.
@wedamija wedamija requested review from a team as code owners April 4, 2025 17:29
@wedamija wedamija enabled auto-merge (squash) April 4, 2025 17:29
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 4, 2025
@wedamija wedamija merged commit 460b49d into master Apr 4, 2025
48 checks passed
@wedamija wedamija deleted the danf/uptime-move-grouptype-v2 branch April 4, 2025 19:20
@wedamija wedamija added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Apr 7, 2025
@getsentry-bot
Copy link
Contributor

revert failed (conflict? already reverted?) -- check the logs

wedamija added a commit that referenced this pull request Apr 7, 2025
wedamija added a commit that referenced this pull request Apr 8, 2025
…ype.py take 2 (#88821)" (#88991)

This reverts commit 460b49d.

<!-- Describe your PR here. -->
andrewshie-sentry pushed a commit that referenced this pull request Apr 8, 2025
…ake 2 (#88821)

We want to keep grouptypes in the module that is related to them, so
just moving this out.

This caused pickling problems last time, so importing the model in its
old location to prevent issues.
andrewshie-sentry pushed a commit that referenced this pull request Apr 8, 2025
…ype.py take 2 (#88821)" (#88991)

This reverts commit 460b49d.

<!-- Describe your PR here. -->
Christinarlong pushed a commit that referenced this pull request Apr 10, 2025
…ype.py take 2 (#88821)" (#88991)

This reverts commit 460b49d.

<!-- Describe your PR here. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert Add to a merged PR to revert it (skips CI)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants