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

[feat]: SQlite #8

Closed
dhruvinsh opened this issue Jan 9, 2024 · 9 comments
Closed

[feat]: SQlite #8

dhruvinsh opened this issue Jan 9, 2024 · 9 comments

Comments

@dhruvinsh
Copy link

Good cli app concept.

SQlite support would be good as well as future development.

@MaxThom
Copy link

MaxThom commented Jan 11, 2024

+1, ive been trying to find a good TUI for SQL and no success. Your lazysql look stunning already.
I'm using sqlite3 and postgres

@jorgerojas26
Copy link
Owner

Hello,

support for SQLite is a work in progress. I want Lazysql to work on many different RDBMS.

PRs are welcome, we can make this tool the best SQL client for the terminal.

@GusJelly
Copy link
Contributor

I'm willing to work on the sqlite stuff, I would like to ask what would be needed to do to add support. I am reading the code right now and I can see that would need to add an sqlite.go file with code that supports the driver. Is there anything else that would need to be done besides that?

@MaxThom
Copy link

MaxThom commented Jan 12, 2024

Likewise, I could try the Postgres implementation!

@jorgerojas26
Copy link
Owner

I am working on the code refactor to support multiple drivers, it should be ready today, maybe tomorrow. With those changes, supporting more drivers will be easy. I'll let you know when it's ready so we all can start adding drivers.

@jaymorelli96
Copy link

Cool!! I am very excited about this project, great work @jorgerojas26. I would also be happy to help with implementation or documentation 😄

@GusJelly
Copy link
Contributor

Thank you for your work @jorgerojas26. Excited for this amazing software, I'll sync my fork and start work on the sqlite driver.

@jorgerojas26
Copy link
Owner

I am working on the code refactor to support multiple drivers, it should be ready today, maybe tomorrow. With those changes, supporting more drivers will be easy. I'll let you know when it's ready so we all can start adding drivers.

I just merged a PR with these changes and added Postgres support. Next will be SQLite.

@GusJelly Do you want to work on the SQLite implementation?

@GusJelly
Copy link
Contributor

I am working on the code refactor to support multiple drivers, it should be ready today, maybe tomorrow. With those changes, supporting more drivers will be easy. I'll let you know when it's ready so we all can start adding drivers.

I just merged a PR with these changes and added Postgres support. Next will be SQLite.

@GusJelly Do you want to work on the SQLite implementation?

Going to sync to start working on it now. Going to read the refactor to see what I need to do.

LostbBlizzard pushed a commit to LostbBlizzard/lazysql that referenced this issue Jul 21, 2024
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

No branches or pull requests

5 participants