Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

Smart completions #85

Merged
merged 10 commits into from
May 3, 2017
Merged

Smart completions #85

merged 10 commits into from
May 3, 2017

Conversation

tkopets
Copy link
Collaborator

@tkopets tkopets commented May 3, 2017

New "smarter" completions are shown, when possible.
The main difference from previous completions are proper completions for table columns: table.column|, as well as completions for table aliases:
select a.| from tbl a
and completions for join conditions:
select * from tbl_a a inner join tbl_b b on |
The current paragraph of SQL text is parsed by already existing sqlparse library.
A small demo can be seen in discussion #67

@tkopets tkopets requested a review from mtxr May 3, 2017 10:03
@mtxr
Copy link
Owner

mtxr commented May 3, 2017

Go head. Can you release the new version?

@mtxr mtxr merged commit 5b37c84 into master May 3, 2017
@tkopets
Copy link
Collaborator Author

tkopets commented May 3, 2017

Sure, a bit later, as I want to fix few things before release.
Right now we have a problem when existing users most probably will not get the updated queries for different RDBMS as Sublime copies the entire contents of SQLTools.sublime-settings into Users directory file after plugin install, so the updated queries most probably will be overridden by queries defined in Users\SQLTools.sublime-settings.
Any ideas on that topic?

@tkopets
Copy link
Collaborator Author

tkopets commented May 3, 2017

I also want to write a short PackageContol update message

@mtxr
Copy link
Owner

mtxr commented May 3, 2017 via email

@tkopets
Copy link
Collaborator Author

tkopets commented May 3, 2017

Do you have any ideas how to force Sublime not to copy entire SQLTools.sublime-settings with contents, but to start with empty config in Users\SQLTools.sublime-settings like this:

{
    
}

Or is there anything we can alter in SQLTools project structure to achieve this?

@tkopets
Copy link
Collaborator Author

tkopets commented May 3, 2017

I think I found a way to do so.

@tkopets tkopets deleted the smart_completions branch May 12, 2017 14:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants