-
Notifications
You must be signed in to change notification settings - Fork 188
Reorganize PostgreSQL section in language_servers.md #2675
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
Conversation
✅ Deploy Preview for sublime-lsp ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docs/src/language_servers.md
Outdated
| ### PostgresSQL | ||
| 1. Install the [Postgres Language Server](https://pg-language-server.com/latest/manual_installation/). | ||
| 2. Open any .sql file and select `View > Syntax > Open all with current extension as... > SQL > PostgreSQL` to set the PostgreSQL syntax as the default. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this is necessary. It seems like PostgreSQL files can have special file extensions too: https://github.com/sublimehq/Packages/blob/9de8e396796f5013ac5dd51a189d4c4bb6537a5d/SQL/PostgreSQL.sublime-syntax#L9-L11
Perhaps just an advice like "Ensure that the PostgreSQL syntax is applied to the relevant files, for example by selecting View > Syntax > SQL > PostgreSQL." would be better.
| "jag": { | ||
| "enabled": true, | ||
| "command": ["jag" "lsp"], | ||
| "command": ["jag", "lsp"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is an unrelated minor change
Moving the PostgreSQL docs to a new SQL section.