-
-
Notifications
You must be signed in to change notification settings - Fork 264
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
Write more precise db schema check #1202
Write more precise db schema check #1202
Conversation
Just to make sure it's clear, this is pointing to |
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.
Nice!
56e5b09
to
a9346f4
Compare
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.
Nice work on this @gazayas! I'm gonna merge it, and then I think that depfu
will notice the update on the rails upgrade PR, and then if tests pass I think it'll automatically merge it.
Soooo....
https://butt.holdings/
a9346f4
to
5d513f2
Compare
003e058
into
depfu/update/group/rails-7.1.2
Again this is failing due to our current CI credit status, but the db schema check works for me. I wrote a comment in #1150 explaining a little bit in more detail.