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(opts): failing to opt-out features (completion and diagnostics) #83

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

atusy
Copy link
Contributor

@atusy atusy commented Jan 22, 2024

Following lines become true even if the specified values are false.
This PR fixes it.

  completion = completion or true
  diagnostics = diagnostics or true

Now, nil and true are treated as true and false is treated as false

@jmbuhr
Copy link
Owner

jmbuhr commented Jan 22, 2024

Good catch, thanks!

@jmbuhr jmbuhr merged commit 0eeb4f9 into jmbuhr:main Jan 22, 2024
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.

2 participants