Add collation completions and more charset completions by rolandwalker · Pull Request #1711 · dbcli/mycli · GitHub
Skip to content

Add collation completions and more charset completions#1711

Merged
rolandwalker merged 1 commit intomainfrom
RW/collation-and-charset-completion-improvements
Mar 16, 2026
Merged

Add collation completions and more charset completions#1711
rolandwalker merged 1 commit intomainfrom
RW/collation-and-charset-completion-improvements

Conversation

@rolandwalker
Copy link
Copy Markdown
Contributor

@rolandwalker rolandwalker commented Mar 14, 2026

Description

Complete in these positions:

  • "string" COLLATE ^
  • CONVERT("string" USING ^
  • CAST("string" as CHAR CHARACTER SET ^

Refactor stored charsets to not be per-schema, which was not necessary.

Add commentary on the pre-existing issue of the WHERE logic short- circuiting other useful completions, and some other edge cases such as overenthusiastic blocking of numeric completions.

Examples:

last image Screenshot 2026-03-14 at 10 02 38 AM

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 .

@rolandwalker rolandwalker self-assigned this Mar 14, 2026
@github-actions
Copy link
Copy Markdown

@rolandwalker rolandwalker force-pushed the RW/collation-and-charset-completion-improvements branch from 256d614 to 1001ed8 Compare March 14, 2026 15:47
@rolandwalker rolandwalker force-pushed the RW/collation-and-charset-completion-improvements branch from 1001ed8 to ab90743 Compare March 14, 2026 17:10
Complete in these positions:

 * "string" COLLATE ^
 * CONVERT("string" USING ^
 * CAST("string" as CHAR CHARACTER SET ^

Refactor stored charsets to not be per-schema, which was not necessary.

Add commentary on the pre-existing issue of the WHERE logic short-
circuiting other useful completions, and some other edge cases such
as overenthusiastic blocking of numeric completions.
@rolandwalker rolandwalker force-pushed the RW/collation-and-charset-completion-improvements branch from ab90743 to 63507be Compare March 16, 2026 09:48
@rolandwalker rolandwalker merged commit 724182f into main Mar 16, 2026
8 checks passed
@rolandwalker rolandwalker deleted the RW/collation-and-charset-completion-improvements branch March 16, 2026 09:53
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