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

Validating CLI arguments #2

Merged
merged 8 commits into from
Jun 1, 2023
Merged

Validating CLI arguments #2

merged 8 commits into from
Jun 1, 2023

Conversation

tl3119
Copy link
Owner

@tl3119 tl3119 commented Jun 1, 2023

Describe your changes

Issue number

Closes #X

Checklist before requesting a review

tonykploomber and others added 8 commits May 31, 2023 21:06
* add in live

* trigger rebuild

* trigger rebuild

* fix syntax

* add password

* only executed by workflow_call

* comment out explicitl kv

* clean up
* --with deprecation in CTE

CTE depr

Warning msg, sqlglot

Lint

tests

warn test

conditions for extract

Empty commit

removed with test

exception handling

Fixed tests

Lint

typo error

test

condition for typo

suggestions

plot changes

merge conflict

fix test

Fix test

fix test

tests

list of with

sqlcmd delete

sqlcmd tests

rebase

Fixes

fixed

test fixes

removed test

non-existent tbl Test fix

fix typo test

fix typo test

space removed

delete snippet test

test fix

Docs

docs

toc

changelog modified

args modified

versionchanged added

message changed

tests

sqlcmd added

deprecation warning changed

tests

depr warning

removed if exists

debug prints

added clean_conns

rebase

test fix

rebase issues

revert error msg

test for snippet utils

telemetry removed

Minor changes

Moved string

refactor

plot tests

plot_typo

Renamed test file

Removed line

test fix

test fix

Review comments

added issue numbers

rebase

lint

* added to bar, pie"

* Empty commit
* fixed merge conflicts and added atextit: ready for review

* added verbose argument to close_all() and atexit.register()
@tl3119 tl3119 merged commit f06d7cc into tl3119:master Jun 1, 2023
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.

5 participants