-
Notifications
You must be signed in to change notification settings - Fork 233
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
debug/ developer tooling accomodations #157
Comments
Optionally this loop over command could be incorporated (and refined to match the deployment gathering Regular expressions) into an expanded render command syntax... as per #156 |
Interesting @MACKAT05. Using the Render to verify the sql that will be executed appears to be useful application of the How do you think this pattern will fare for CI/CD based deployment workflows in Development environment? Are you envisioning the iterative development is done locally, say on a clone and then pushed for actual deployment in Development and higher environments? |
@sfc-gh-tmathew i started working on a fork to deal with this and other issues...
Getting to the point of this query I recall seeing somewhere that there was a technical reason for keeping it as one file... does that still need to happen? |
This appears to be a major refactor. Appears to be a 4.0 release candidate. Will get back to you @MACKAT05 once I have a chance to sync up with @jeremiahhansen. |
@MACKAT05 , in #275 I've split the cli.py file out into multiple files / directories to make it easier to write unit tests and such. I poked around your fork and couldn't find a branch with the folder structure above. Is that work in a local branch that you haven't committed? Do you think you could build your feature off of #275? |
yeah sorry about that it is still on my local... i'll try to get that visible and up later tonight. |
Developer tooling.
I wrote a simple PowerShell script to run the render command over all the files and redirect them into a subfolder that is listed in the .gitignore
Can we add Block quotes to the outputs like so?:
This would allow Vs code users to execute and debug the rendered code interactively as captured from the output with the new snowflake extension.
The text was updated successfully, but these errors were encountered: