-
Notifications
You must be signed in to change notification settings - Fork 960
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
Update sl-jdbc.md #3886
Update sl-jdbc.md #3886
Conversation
I realized the examples in the jdbc api query table are missing single quotes! I added them
Hello!👋 Thanks for contributing to the dbt product documentation and opening this pull request! ✨ |
✅ Deploy Preview for docs-getdbt-com ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
remove excess apostrophe
hey @rpourzand thanks for opening this up! i left some questions for other fields as they may not need apostrophes -- wanted to double check with you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @rpourzand , looks good! approving to merge ✨ ! thanks for opening this up!
I realized the examples in the jdbc api query table are missing single quotes! I added them
What are you changing in this pull request and why?
Checklist
Adding new pages (delete if not applicable):
website/sidebars.js
Removing or renaming existing pages (delete if not applicable):
website/sidebars.js
website/static/_redirects