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.

Sign up for GitHub

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

Jump to bottom

Add extra validation for the `rename_constraint` operation #303

Merged

andrew-farries merged 2 commits into main from validation-for-rename-constraint-op

Mar 4, 2024

Collaborator

andrew-farries commented Mar 4, 2024

Add an extra step to the validation for the rename_constraint operation to ensure that the referenced table exists.
Add validation tests for the operation.

Fixes #302

andrew-farries added 2 commits

March 4, 2024 12:27


          Add validation tests for rename constraint op

f1c34d7


          Validate table existence in rename constraint op

3dd0b4e

Avoid a nil reference panic if the table doesn't exists.

andrew-farries requested review from exekias and SferaDev

March 4, 2024 12:34

SferaDev approved these changes

View reviewed changes

andrew-farries merged commit fb05aec into main

42 checks passed

andrew-farries deleted the validation-for-rename-constraint-op branch

March 4, 2024 13:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment