Skip to content

feat(build): add negative build flags to compile squix without a set of database drivers - #91

Merged
eduardofuncao merged 1 commit into
mainfrom
driver-flags
Sep 22, 2026
Merged

eduardofuncao merged 1 commit into
mainfrom
driver-flags

Conversation

@eduardofuncao

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI 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.

Copilot review overview

🟡 Changes recommended

Moderate issues remain in CGO/negative-tag reporting, local full builds, and error propagation.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 2 Medium severity

Open (2)
What changed in this PR

Adds build-time database-driver variants and runtime reporting of compiled driver support.

Changes:

  • Adds per-driver negative build tags and conditional registration.
  • Adds full, lite, and minimal build targets across local, Nix, and release workflows.
  • Adds squix --drivers and documents driver availability.
File Reviewed change
README.md Links to build variant documentation.
Makefile Adds lite and minimal targets.
internal/​db/​sqlserver_connection.go Adds conditional SQL Server driver compilation.
internal/​db/​sqlite_connection.go Adds conditional SQLite driver compilation.
internal/​db/​snowflake_connection.go Adds conditional Snowflake driver compilation.
internal/​db/​postgres_connection.go Adds conditional PostgreSQL driver compilation.
internal/​db/​oracle_connection.go Adds conditional Oracle driver compilation.
internal/​db/​mysql_connection.go Adds conditional MySQL driver compilation.
internal/​db/​inference.go Reports compiled drivers.
internal/​db/​firebird_connection.go Adds conditional Firebird driver compilation.
internal/​db/​duckdb_connection.go Adds conditional DuckDB driver compilation.
internal/​db/​duckdb_connection_stub.go Removes the obsolete DuckDB stub.
internal/​db/​connection_factory.go Centralizes conditional driver creation.
internal/​db/​clickhouse_connection.go Adds conditional ClickHouse driver compilation.
flake.nix Adds Nix build variants.
docs/​features.md Documents variants and driver coverage.
cmd/​squix/​drivers.go Adds driver availability output.
cmd/​squix/​app.go Registers the --drivers command.
build-local.sh Builds local variants.
.github/​workflows/​release.yml Builds and publishes variant releases.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread cmd/squix/drivers.go
Comment thread internal/db/connection_factory.go
@eduardofuncao
eduardofuncao merged commit c8583cb into main Sep 22, 2026
19 checks passed
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.

2 participants