Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

%sqlcmd improvements #341

Merged
merged 13 commits into from
Apr 18, 2023

Conversation

gtduncan
Copy link

@gtduncan gtduncan commented Mar 30, 2023

Describe your changes

Improving user experience for %sqlcmd, more specific errors

Issue number

Closes #275

Checklist before requesting a review


📚 Documentation preview 📚: https://jupysql--341.org.readthedocs.build/en/341/

@edublancas edublancas changed the title %sqlcmb improvements %sqlcmd improvements Mar 30, 2023
@gtduncan
Copy link
Author

gtduncan commented Apr 4, 2023

Functionality + documentation should be all finished, only thing I'm still having trouble implementing is using PrettyTable for the failing test cases... will keep looking into it.

@gtduncan gtduncan marked this pull request as ready for review April 4, 2023 16:33
@gtduncan gtduncan requested a review from tonykploomber April 5, 2023 17:57
@tonykploomber
Copy link

Resolve the conflict?

# This is the 1st commit message:

initial commit

# This is the commit message ploomber#2:

comparator check

# This is the commit message ploomber#3:

remove test

# This is the commit message ploomber#4:

temp

temp

column not found

format

# This is the commit message ploomber#5:

edits

remove comments

concise

remove comments

# This is the commit message ploomber#6:

print values
comparator check

remove test

temp

temp

column not found

format

edits

remove comments

concise

remove comments

print values

integration tests fix

lint

added quotes for test

integration test fix

fix

docs fix

documentation, changelog
@gtduncan gtduncan force-pushed the george-sqlcmb-improvements branch from 7d234f1 to 7d7ed83 Compare April 12, 2023 16:08
@gtduncan
Copy link
Author

Should be resolved now

@gtduncan gtduncan added allow-workflow-edits Allow edits to .github/workflows and removed allow-workflow-edits Allow edits to .github/workflows labels Apr 13, 2023
tonykploomber
tonykploomber previously approved these changes Apr 13, 2023
Copy link

@tonykploomber tonykploomber left a comment

Choose a reason for hiding this comment

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

LG

Copy link

@idomic idomic left a comment

Choose a reason for hiding this comment

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

Please fix

Copy link

@idomic idomic left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

@tonykploomber tonykploomber left a comment

Choose a reason for hiding this comment

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

LG

@tonykploomber tonykploomber merged commit b3ae7a5 into ploomber:master Apr 18, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fixes to %sqlcmd test
4 participants