Skip to content

feat: add SQLite source connector - #146

Merged
masukai merged 1 commit into
drt-hub:mainfrom
PFCAaron12:git-checkout--b-feat/sqlite-source-connector
Apr 2, 2026
Merged

feat: add SQLite source connector#146
masukai merged 1 commit into
drt-hub:mainfrom
PFCAaron12:git-checkout--b-feat/sqlite-source-connector

Conversation

@PFCAaron12

Copy link
Copy Markdown
Collaborator

What does this PR do?

-add SQLite source connector
-updated the drt/config/credentials.py file
-created unit tests and all passed successfully
-created a sqlite to google sheets example to resemble duckdb to google sheets example as close as possible for consistency

Related Issue

Closes #90

Checklist

  • Tests pass (make test)
  • Linter passes (make lint)
  • Updated CHANGELOG.md (if user-facing change)

@masukai masukai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎉 Great first contribution — SQLite source with zero extra dependencies is a nice addition!

I'll clean up a few minor things after merge (trailing whitespace, duplicate import, missing newlines). No action needed from your side.

Thanks! If you enjoy using drt, a ⭐ would be appreciated 🙏

@masukai
masukai merged commit 8771960 into drt-hub:main Apr 2, 2026
masukai added a commit that referenced this pull request Apr 2, 2026
- Remove duplicate sqlite3 import in test_connection
- Fix trailing whitespace in credentials.py and sqlite.py
- Add missing trailing newlines
- Remove trailing spaces from test_connection lines

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
masukai added a commit that referenced this pull request Apr 2, 2026
The #146 merge added SQLiteProfile to credentials.py but didn't
update _get_source in cli/main.py, causing mypy errors in both
cli/main.py and mcp/server.py.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

good first issue: add SQLite source connector

2 participants