Skip to content

Comments

Add SQL statement generator#107

Closed
worksofliam wants to merge 1 commit intomainfrom
feature/sql_generator
Closed

Add SQL statement generator#107
worksofliam wants to merge 1 commit intomainfrom
feature/sql_generator

Conversation

@worksofliam
Copy link
Member

Introduce a SQL statement generator that creates and writes SQL scripts for resolved objects, dependencies, and symbols. Update CLI to support SQL file generation.

Signed-off-by: worksofliam <mrliamallan@live.co.uk>
Copy link
Member

@SanjulaGanepola SanjulaGanepola left a comment

Choose a reason for hiding this comment

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

Nice work!

When running against company system though, I did notice that the same dependency was added twice. This looks to be unrelated to this change as it happens for the other bf options, but it would break this sql script. I believe it is a casing issue. Should I create a new issue?
image

@worksofliam
Copy link
Member Author

@SanjulaGanepola yes, a new issue with how to recreate it will be important! We should have a test case to cover this.

@worksofliam worksofliam marked this pull request as draft April 1, 2025 19:06
@worksofliam
Copy link
Member Author

@SanjulaGanepola I changed my mind. I think I should fix that inside of this PR.

@worksofliam worksofliam self-assigned this Apr 1, 2025
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.

2 participants