Skip to content
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

[Feature] Add support for Procedures #185

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

bplunkett-stripe
Copy link
Collaborator

@bplunkett-stripe bplunkett-stripe commented Dec 11, 2024

Description

Add support for procedures. This support is very --iffy-- because Postgres does not actually track procedure dependencies outside of at creation time. In other words, you can drop a procedure's dependencies without dropping the procedure.

Motivation

Closes #159

Testing

Tested via acceptance tests

@bplunkett-stripe bplunkett-stripe changed the title Bplunkett/add proc support [Feature] Add support for Procedures Dec 11, 2024
@bplunkett-stripe bplunkett-stripe force-pushed the bplunkett/add-proc-support branch from 0e7a920 to 1d3feb9 Compare December 11, 2024 00:53
@bplunkett-stripe bplunkett-stripe marked this pull request as ready for review December 11, 2024 00:53
@bplunkett-stripe bplunkett-stripe force-pushed the bplunkett/add-proc-support branch from 1d3feb9 to f1c96e0 Compare December 11, 2024 01:01
@bplunkett-stripe bplunkett-stripe force-pushed the bplunkett/add-proc-support branch from f1c96e0 to 8d891ea Compare December 11, 2024 01:06
@@ -0,0 +1,108 @@
package diff
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is identical to the previous sql generator. It has just moved.

@bplunkett-stripe bplunkett-stripe merged commit 72e3df5 into main Dec 11, 2024
9 checks passed
@bplunkett-stripe bplunkett-stripe deleted the bplunkett/add-proc-support branch December 11, 2024 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for stored procedures
1 participant