-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
ref(sentry_apps): move sentryapp model to sentry_apps dir #77521
ref(sentry_apps): move sentryapp model to sentry_apps dir #77521
Conversation
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## master #77521 +/- ##
==========================================
+ Coverage 78.08% 78.12% +0.04%
==========================================
Files 6973 6958 -15
Lines 309431 308884 -547
Branches 50655 50548 -107
==========================================
- Hits 241610 241325 -285
+ Misses 61398 61156 -242
+ Partials 6423 6403 -20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete apps.py shim so that we won't get circular import errors from django 🌵
Literally hero level work, thank you. |
Let's start migrating Sentry App resources starting witht he SentryApp model & test !
ref(#73857)
relies on (#77614) & (https://github.com/getsentry/getsentry/pull/15146)