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

release: merge staging into master #1031

Closed
wants to merge 58 commits into from
Closed

release: merge staging into master #1031

wants to merge 58 commits into from

Conversation

gaurav274
Copy link
Member

No description provided.

gaurav274 and others added 30 commits August 28, 2023 23:56
Co-authored-by: Joy Arulraj <[email protected]>
Co-authored-by: Joy Arulraj <[email protected]>
1. Fix all BUILD warning.
2. Fix all links.
3. Add circle ci test for Doc build and link validation.

Documentation content updates will be in the following PRs.

---------

Co-authored-by: Gaurav Tarlok Kakkar <[email protected]>
Co-authored-by: Joy Arulraj <[email protected]>
Co-authored-by: Joy Arulraj <[email protected]>
jarulraj and others added 27 commits August 30, 2023 23:07
* verify connection while creating database
* verbose msg for duplicate database

---------

Co-authored-by: Gaurav Tarlok Kakkar <[email protected]>
1. Database handler `get_columns` also returns the column datatypes
2. Throw Import errors instead of auto-installing missing dependencies.
…ls (#1008)

Rollback the catalog entry if the create query fails
We enable the unit test and short integration test coverage for open
PRs.
Temporarily directly add notebook into usecase section of documentation
to make sure the link on README.md is not broken.
Inspired by https://ludwig.ai/latest/getting_started/.

Topics will be covered in this PR: 

- Installation Guide
- Integrate Data Source
- USE SQL API
- CREATE DATABASE SQL API
* For Postgres CI, add cmd to install packages
* Ignore CI for notebook 14
* It could work with Postgres CI, but we don't have an OpenAI key setup
for CI. It still will fail in the end.
- Integrated mysql data source
Co-authored-by: Rohith Mulumudy <[email protected]>
This diff adds extends the database interface to add support for SQLite.

Test Plan : UT

---------

Co-authored-by: Ankith Reddy Chitti <[email protected]>
Co-authored-by: Ankith Reddy Chitti <[email protected]>
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@gaurav274 gaurav274 closed this Sep 2, 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.

6 participants