Skip to content

docs: add missing libSQL row to test-database table - #1637

Merged
cevheri merged 2 commits into
libredb:mainfrom
Furox-Art:gfi-libredb-libsql-row
Oct 10, 2026
Merged

cevheri merged 2 commits into
libredb:mainfrom
Furox-Art:gfi-libredb-libsql-row

Conversation

@Furox-Art

Copy link
Copy Markdown
Contributor

Summary

Closes #1265.

Two changes to the local test-database table in docs/providers/README.md, both from the issue:

  1. libSQL row added. | libSQL | \libsql` | localhost | 18080 | none | none | none | — |, placed after the Redis row and before Couchbase, matching the order the paragraph above the table counts the engine containers in. Port 18080is bold like OpenSearch's9201, because it is not the engine's usual port: database-compose.ymlmaps the container's8080to18080so it does not collide with Trino. No user, password or token, no TLS, no profile — the compose service carries noprofiles:key anddocs/providers/libsql.md` describes the auth-free local mode.

  2. *none* in the Profile column replaced with the em dash the PostgreSQL row uses. The issue named the Prometheus, Apache Kafka, etcd and Neo4j rows (lines 284-287 of the pinned commit). Since then main has gained more rows carrying the same wrong placeholder — InfluxDB 1.x, InfluxDB 2.x, InfluxDB 3 (SQL), Oxia, Milvus and Qdrant — so all ten are fixed here, which is what the "Done when" criterion ("every row that needs no profile shows the same placeholder") asks for. *none* is defined under the table as "leave the field empty", which cannot apply to a column that is not a connection field.

Only docs/providers/README.md changes. The User, Password and Database columns keep their *none* placeholders — those are connection fields and the placeholder is correct there.

Verification

  • git diff --stat: one file, 11 insertions, 10 deletions.
  • Every changed Profile cell was checked against the base table: the em dash is the placeholder used by PostgreSQL, MySQL, Oracle, Db2, SQL Server, MongoDB, Redis, Couchbase, ClickHouse, Elasticsearch, OpenSearch, Trino, Databend and Cassandra, i.e. every row that needs no profile.
  • The libSQL values are read off database-compose.yml (service libsql, port mapping 18080:8080, no profiles: key) and docs/providers/libsql.md (auth-free local mode, no TLS) rather than assumed.
  • Markdown table column count checked per row: eight cells, matching the header.
  • No test or lint gate applies to a docs-only change; Lint, Typecheck and Build and Unit & Integration Tests are left to run on the PR.

@cevheri
cevheri merged commit bd722f3 into libredb:main Oct 10, 2026
3 checks passed
@cevheri

cevheri commented Oct 10, 2026

Copy link
Copy Markdown
Member

thanks and welcome
how was your first contribution on libredb studio
which database are you using in libredb studio

@cevheri cevheri added the documentation Improvements or additions to documentation label Oct 10, 2026
@Furox-Art

Copy link
Copy Markdown
Contributor Author

Thanks for the warm welcome and for merging my PR!

I'm a self-taught developer, so it was nice to contribute to the project. I haven't actually used LibreDB Studio yet. I came across the documentation issue, saw that I could help, and decided to fix it.

Glad I could contribute, and thanks again!

@cevheri

cevheri commented Oct 11, 2026

Copy link
Copy Markdown
Member

thanks
so you can try it and you can share your taughts :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs/providers/README.md: add the missing libSQL row to the test-database table

2 participants