-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
ref(workflow_engine): Drop workflow_engine.workflowaction table #78277
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
base: master
Are you sure you want to change the base?
Conversation
This PR has a migration; here is the generated SQL for --
-- Raw SQL operation
--
DROP TABLE "workflow_engine_workflowaction"; |
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #78277 +/- ##
===========================================
+ Coverage 61.67% 78.11% +16.44%
===========================================
Files 7059 7068 +9
Lines 311465 311799 +334
Branches 50912 50957 +45
===========================================
+ Hits 192081 243574 +51493
+ Misses 108141 56466 -51675
- Partials 11243 11759 +516 |
ddbbb65
to
2532aa4
Compare
🔒 holding off on merging this until the deploy for the first part is complete (requirement according to https://develop.sentry.dev/development/database-migrations/#deleting-tables) |
This PR has a migration; here is the generated SQL for --
-- Raw SQL operation
--
DROP TABLE "workflow_engine_workflowaction"; |
2532aa4
to
dc63873
Compare
dc63873
to
6b9a7b4
Compare
6b9a7b4
to
aff15de
Compare
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you add the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
…reduce migration churn a little
…reduce migration churn a little
Description
Follow up PR for: #78272
Will complete the removal of the table 🎉