Replace Ambiguous language for tests from switch to parameter for accuracy #10666
Conversation
There was a problem hiding this comment.
The more proper term would be parameter rather than argument
|
Hey Sean, I thought the same thing at first. However after looking deeply at the terminology. I decided the term "argument" would be better. Because "parameter" is the "-value" or $variable that gets defined when the argument is passed into the command. https://www.quora.com/What-is-the-difference-between-argument-and-parameters-in-C Is there an official consensus that the PowerShell team uses over the conventional comp sci answer?
|
|
@romero126 In PowerShell and in this example |
|
I agree with @sdwheeler here. |
|
Thank you @sdwheeler and @SteveL-MSFT for the feedback. I would have replied sooner but I got a little busy. |
|
@sdwheeler Please update your review. |
|
@romero126 Thanks for your contribution! |
|
🎉 Handy links: |
PR Summary
Replace Ambiguous language for pester tests.
Pester tests for Remove-Item used incorrect terminology using switch analogous with Parameter
PR Context
Modify incorrect terminology, to use
parameterinstead ofswitch.PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.