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

fix: coalesce nil start line to 1 #182

Merged

Conversation

dvic
Copy link
Contributor

@dvic dvic commented Aug 14, 2023

Fixes #160

@dvic dvic force-pushed the 2023-08-14-fix-line-null-constraint branch from 0276343 to 4324b42 Compare August 14, 2023 20:41
lib/next_ls/runtime/sidecar.ex Outdated Show resolved Hide resolved
lib/next_ls/runtime/sidecar.ex Outdated Show resolved Hide resolved
@dvic dvic force-pushed the 2023-08-14-fix-line-null-constraint branch 2 times, most recently from 1c566ff to c98fec1 Compare August 14, 2023 21:09
@dvic dvic marked this pull request as ready for review August 14, 2023 21:10
@dvic dvic force-pushed the 2023-08-14-fix-line-null-constraint branch 3 times, most recently from e143e24 to b355a62 Compare August 14, 2023 21:17
@mhanberg mhanberg force-pushed the 2023-08-14-fix-line-null-constraint branch from b355a62 to 9c4e1fc Compare August 14, 2023 22:00
@mhanberg mhanberg enabled auto-merge (squash) August 14, 2023 22:00
@mhanberg mhanberg merged commit 9864fc4 into elixir-tools:main Aug 14, 2023
3 checks passed
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.

[NextLS] sqlite3 error: NOT NULL constraint failed: references.start_line
2 participants