-
-
Notifications
You must be signed in to change notification settings - Fork 320
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
Replace column index usage in db column alter operations with column attnum. #1132
Conversation
Fix `alter_column_type` function call to use table oid in `infer_column_type`
…tnum-column-alter
…calling `alter_column_type`
Codecov Report
@@ Coverage Diff @@
## master #1132 +/- ##
=======================================
Coverage 93.37% 93.38%
=======================================
Files 112 112
Lines 4286 4291 +5
=======================================
+ Hits 4002 4007 +5
Misses 284 284
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Looks good. One minor change requested.
…' into mathesar-993-attnum-column-alter
Fixes #993
Technical details
Checklist
Update index.md
).master
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin