Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(extension): quote cmd on windows #4988

Merged
merged 1 commit into from
Apr 23, 2024
Merged

fix(extension): quote cmd on windows #4988

merged 1 commit into from
Apr 23, 2024

Conversation

fannheyward
Copy link
Member

Closes #4987

on windows, command should be quoted if containing spaces. nodejs/node#7367

Closes #4987

on windows, command should be quoted if containing spaces. nodejs/node#7367
Copy link

codecov bot commented Apr 22, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 98.55%. Comparing base (77ae2d4) to head (7adc95a).

Files Patch % Lines
src/extension/installer.ts 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4988   +/-   ##
=======================================
  Coverage   98.55%   98.55%           
=======================================
  Files         273      273           
  Lines       26070    26071    +1     
  Branches     5389     5390    +1     
=======================================
+ Hits        25692    25694    +2     
+ Misses        222      220    -2     
- Partials      156      157    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fannheyward fannheyward merged commit 2ec3166 into master Apr 23, 2024
3 of 4 checks passed
@fannheyward fannheyward deleted the fix/spawn-win32 branch April 23, 2024 01:10
fannheyward added a commit that referenced this pull request Apr 23, 2024
2ec3166 fix(extension): quote cmd on windows (#4988)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't install any plugins if node is installed inside a folder with a space on Windows
1 participant