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

[NextLS] sqlite3 error: NOT NULL constraint failed: references.start_line #160

Closed
ndan opened this issue Aug 11, 2023 · 3 comments · Fixed by #182 or #169
Closed

[NextLS] sqlite3 error: NOT NULL constraint failed: references.start_line #160

ndan opened this issue Aug 11, 2023 · 3 comments · Fixed by #182 or #169

Comments

@ndan
Copy link

ndan commented Aug 11, 2023

I've installed elixir-tools.vscode from the marketplace, and it's working, but there are tons of errors in the NextLS output.

statement: INSERT INTO 'references' (identifier, arity, file, type, module, start_line, start_column, end_line, end_column)
    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?);
[Error - 10:12:27 AM] [NextLS] sqlite3 error: NOT NULL constraint failed: references.start_line

elixir-tools: 0.6.0
elixir: 1.15.4-otp-26
erlang: 26.0.2
OS: MacOS Ventura (13.4.1)
VSCode: 1.81.0

@mhanberg
Copy link
Collaborator

Thanks for the detailed report!

I've noticed these as well, it shouldn't effect anything really, unless you notice poor results in the Find References feature.

It should probably say warn instead of error.

Let me know if you see any inaccuracies there.

@mhanberg
Copy link
Collaborator

I should clarify that these are alarming and I will look into them, but the presence of them means the LSP should still be functional rather than broken.

@mhanberg
Copy link
Collaborator

I think for these, these are functions that are generated by elixir and don't have a spot in the source code, so we coalesce the nil line into 1 (or 0, whatever it is supposed to be)

dvic added a commit to dvic/next-ls that referenced this issue Aug 14, 2023
dvic added a commit to dvic/next-ls that referenced this issue Aug 14, 2023
dvic added a commit to dvic/next-ls that referenced this issue Aug 14, 2023
dvic added a commit to dvic/next-ls that referenced this issue Aug 14, 2023
dvic added a commit to dvic/next-ls that referenced this issue Aug 14, 2023
dvic added a commit to dvic/next-ls that referenced this issue Aug 14, 2023
dvic added a commit to dvic/next-ls that referenced this issue Aug 14, 2023
mhanberg pushed a commit to dvic/next-ls that referenced this issue Aug 14, 2023
mhanberg pushed a commit that referenced this issue Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment