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 JS falsy comparison #177

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Fix JS falsy comparison #177

merged 1 commit into from
Feb 15, 2024

Conversation

SuperAuguste
Copy link
Collaborator

@SuperAuguste SuperAuguste commented Feb 15, 2024

JavaScript is weird lol

This fixes ZLS not starting automatically. "" as the path means that Zig and/or ZLS will be looked up in PATH as intended, whereas undefined is an undesired state. Interestingly, undefined should never actually occur as "" is the default, so this fix technically does nothing. :P

Not sure if "" to designate looking up by PATH is the best approach, but I'm not sure what the convention is elsewhere.

Closes #176

@SuperAuguste SuperAuguste requested a review from Vexu February 15, 2024 14:23
@SuperAuguste SuperAuguste force-pushed the auguste/js-falsy-noooo branch from 2b496f4 to e3cc045 Compare February 15, 2024 14:48
Copy link
Member

@Vexu Vexu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the fix I was thinking of. If you can confirm this fixes it for you then go ahead with the merge.

@SuperAuguste
Copy link
Collaborator Author

Confirmed locally :)

@SuperAuguste SuperAuguste merged commit f80998e into master Feb 15, 2024
2 checks passed
@SuperAuguste SuperAuguste deleted the auguste/js-falsy-noooo branch February 15, 2024 14:57
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.

The extension no longer automatically starts ZLS
2 participants