Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add FROM to InitialKeywords vector in autocomplete extension #9877

Conversation

hamilton
Copy link
Contributor

@hamilton hamilton commented Dec 3, 2023

This is a small patch to the autocomplete library, adding FROM to the list of expected initial keywords. In other words, if I write FROM sql_auto_complete('FRO') I should expect to get back FROM.

Adds a test case for this in tools/shell/tests/test_autocomplete.py.

I haven't made a patch to this part of duckdb before so please let me know if I've made a mistake!

@hamilton hamilton changed the title add FROM to list of InitialKeywords in autocomplete extension add FROM to InitialKeywords vector in autocomplete extension Dec 3, 2023
@Mytherin Mytherin merged commit b53995d into duckdb:main Dec 4, 2023
43 checks passed
@Mytherin
Copy link
Collaborator

Mytherin commented Dec 4, 2023

Thanks! LGTM

krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Dec 14, 2023
Merge pull request duckdb/duckdb#9872 from Tishj/regexp_matches_null_optimizer
Merge pull request duckdb/duckdb#9877 from hamilton/hamilton/autocomplete-add-from-to-initial-keywords
Merge pull request duckdb/duckdb#9875 from Mytherin/progressbardisable
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.

None yet

2 participants