-
Notifications
You must be signed in to change notification settings - Fork 4
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
Drop unused roles and tables #134
Conversation
🎉 All dependencies have been resolved ! |
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.
@thomasyu888 this is ready to merge, but I'll wait until the most recent dev
is rebased onto main
because there is a V script in here I'd like to verify after it's merged into dev
. Thanks for working on this with me!
@jaymedina can you double check, I merged everything from |
@thomasyu888 there is one straggler commit here that I'd like to see in |
@jaymedina Ah, thanks for the clarification. I created this PR: #138 |
51153d6
to
e2ddf2f
Compare
LGTM! Feel free to merge @philerooski |
e2ddf2f
to
01571cd
Compare
|
We no longer have a need for these database roles or tables.
Depends on #133