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

Support more valid shell names on Windows #1137

Merged
merged 7 commits into from Aug 20, 2023
Merged

Support more valid shell names on Windows #1137

merged 7 commits into from Aug 20, 2023

Conversation

ericcornelissen
Copy link
Owner

Relates to #1125

Summary

Update how shell names for Windows are handled in order to support shell names specified without extension (.exe).

@ericcornelissen ericcornelissen added the enhancement New feature or request label Aug 20, 2023
@github-actions github-actions bot added ci/cd Relates to ci/cd test Relates to testing labels Aug 20, 2023
@codecov
Copy link

codecov bot commented Aug 20, 2023

Codecov Report

Merging #1137 (58672a4) into main (6940e0f) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main     #1137   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          709       709           
=========================================
  Hits           709       709           
Flag Coverage Δ
integration-MacOS 93.65% <0.00%> (ø)
integration-Ubuntu 93.65% <0.00%> (ø)
integration-Windows 92.24% <100.00%> (ø)
unit 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
src/win.js 100.00% <100.00%> (ø)

@ericcornelissen ericcornelissen merged commit e9fe111 into main Aug 20, 2023
40 checks passed
@ericcornelissen ericcornelissen deleted the win-no-exe branch August 20, 2023 13:31
@ericcornelissen ericcornelissen changed the title Support Windows shell without .exe included Support more valid shell names on Windows Aug 20, 2023
@ericcornelissen ericcornelissen removed the ci/cd Relates to ci/cd label Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request test Relates to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant