Require sqlglot 29.x with C extensions by rolandwalker · Pull Request #1707 · dbcli/mycli · GitHub
Skip to content

Require sqlglot 29.x with C extensions#1707

Merged
rolandwalker merged 1 commit intomainfrom
RW/require-sqlglot-29
Mar 13, 2026
Merged

Require sqlglot 29.x with C extensions#1707
rolandwalker merged 1 commit intomainfrom
RW/require-sqlglot-29

Conversation

@rolandwalker
Copy link
Copy Markdown
Contributor

Description

Require sqlglot 29.x with C extensions, since the Rust extensions are now deprecated.

Fixes #1706. Seems that we can only commit to going forward. Is there a pyproject.toml syntax which would allow different extras for 28.x vs 29.x?

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

since the Rust extensions are now deprecated
@rolandwalker rolandwalker self-assigned this Mar 13, 2026
@github-actions
Copy link
Copy Markdown

@rolandwalker rolandwalker merged commit 21a6314 into main Mar 13, 2026
10 checks passed
@rolandwalker rolandwalker deleted the RW/require-sqlglot-29 branch March 13, 2026 17:37
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.

mycli started to show a warning about sqlglot[rs] on startup.

1 participant