-
Notifications
You must be signed in to change notification settings - Fork 26
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
Can't connect to DB #9
Comments
Hi, thanks for reporting, @narmyondiscord! Added rudimentary logging in output panel (Inline SQL tab) in v2.10.3. Might be related - one upstream issue with sql-lint I observed is that if the first keyword in SQL string is not one of the predefined SQL keywords - it does not run package main
func main() {
query := `--sql
sselect * from book where price = 11;
`
}
|
I'm running into the same issue, didn't know whether I should have created a new issue or comment on this one. My setup uses WSL2 on Windows 11, this extension is being installed on my WSL's In my workspace {
"inlineSQL.dbDriver": "postgres",
"inlineSQL.dbHost": "localhost",
"inlineSQL.dbPort": 5432,
"inlineSQL.dbUser": "docker",
"inlineSQL.dbPassword": "docker",
"inlineSQL.enableDBIntegration": true,
"inlineSQL.lintSQLFiles": false
} In my Javascript code I write: const s = `--sql
SELECT yo FORM order WHERE order_id="hello"
`; Which I know is incorrect. The correct form should be: SELECT * FROM orders WHERE id=1; But instead, all I get in
Is there anything that I might be doing wrong that prevents the linting from working? UPDATE: |
omg, this has taken me on emotional rollecoaster when I discovered the integration and when I found out that there is no way to indicate database name in like 30 seconds after. 😂 |
I have put my database credentials in the settings.json, but it does not seem to connect to my database. Is there any Log?
The text was updated successfully, but these errors were encountered: