Change positional parameter for powershell.exe from -Command to -File#4019
Change positional parameter for powershell.exe from -Command to -File#4019daxian-dbw merged 3 commits intoPowerShell:masterPowerShell/PowerShell:masterfrom SteveL-MSFT:powershell-file-argSteveL-MSFT/PowerShell:powershell-file-argCopy head branch name to clipboard
Conversation
|
@SteveL-MSFT You might find limiting that settting to just PowerShell files to be handy - especially for this repo e.g.: Then VSCode will not mess with trailing whitespace on .cs and .resx files. |
|
You probably should fix the resx file to only include your intended change - the other whitespace changes do have a visible effect in the binary, which is probably fine, but should be in a different commit. In reply to: 308830296 [](ancestors = 308830296) |
There was a problem hiding this comment.
Does this set the exit code correctly? You don't have a test - you should add one as well.
There was a problem hiding this comment.
C# now supports local functions - that would make this a bit simpler because you wouldn't the parameters.
There was a problem hiding this comment.
Making ParseFile a local function would make it inconsistent with similar methods like ParseCommand(), ParseFormat(), ParseExecutionPolicy(), etc...
|
Isn't this a breaking change? It should be documented in the release notes. |
… to execute. To align with POSIX so that things like shebang scripts work correctly, we are changing powershell.exe so that it treats unknown arguments (aka positional argument) as a file. This means that `powershell foo` will now attempt to use `foo` as a PowerShell script whereas previously `foo` would be treated as a command to execute. This doesn't affect existing usage of either `-File` nor `-Command`. Fixed tests that didn't explicitly use `-Command` parameter.
added test to validate exit code from script
18436cb to
a7a2497
Compare
|
@lzybkr any other feedback, would like to get this into beta.3 since it's a breaking change and get the feedback |
Previously powershell.exe treated unknown arguments as a command line to execute. To align with POSIX so that things like shebang scripts work correctly, we are changing powershell.exe so that it treats unknown arguments (aka positional argument) as a file. This means that
powershell foowill now attempt to usefooas a PowerShell script whereas previouslyfoowould be treated as a command to execute. This doesn't affect existing usage of either-Filenor-Command. Fixed tests that didn't explicitly use-Commandparameter.Note that some of the changes below for trailing whitespace was due to VSCode setting.
Fix #1103
Fix #3959