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

fixes to %sqlcmd test #275

Closed
edublancas opened this issue Mar 20, 2023 · 4 comments · Fixed by #341
Closed

fixes to %sqlcmd test #275

edublancas opened this issue Mar 20, 2023 · 4 comments · Fixed by #341
Assignees

Comments

@edublancas
Copy link

edublancas commented Mar 20, 2023

the %sqlcmd test needs improvements in terms of docs and user experience

notebook for context: sql-test.ipynb.zip

@edublancas
Copy link
Author

notes: we're missing the changelog entry and also the notice in the docs that states in which version this was added

@edublancas
Copy link
Author

and this feature deserves its own tutorial, instead of embedding if in the listing tables/columns one

@gtduncan
Copy link

Just to clarify, the contents of that .ipynb should be put/paraphrased in the standalone %sqlcmd tutorial right?

@edublancas
Copy link
Author

so there are two things here: one is fixing the logic in %sqlcmd test according to the comments in the attached notebook.

Once that's done, we need to write a tutorial showing how to use it.

@gtduncan gtduncan mentioned this issue Mar 30, 2023
4 tasks
@sync-by-unito sync-by-unito bot closed this as completed Apr 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants