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

Schema feature #241

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Schema feature #241

wants to merge 2 commits into from

Conversation

Eccenux
Copy link

@Eccenux Eccenux commented Oct 3, 2023

Search path / Schema #155

Don't seem to be working yet, but I think it should. Can someone give some debug/devel pointers?

I've executed Extension launch config and was able to debug selecting schema. And also that worked fine :-)

But I was not able to debug src\language\server.ts. Sorry if that is a stupid question, but does the language server need to be executed separately?

Also, am I even going in the right direction here? I mean would that kind of feature be OK?

@Eccenux
Copy link
Author

Eccenux commented Oct 3, 2023

I was also thinking about using special syntax in the database field to avoid some complications. So instead of introducing new properties just use e.g. mydb.myschema syntax. So only DB selector and language server would need to be changed then. Added benefit would be that the schema would already be displayed in GUI (in the status bar).

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

1 participant