| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
The more proper term would be parameter rather than argument
Sorry, something went wrong.
|
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?
|
Sorry, something went wrong.
|
romero126 In PowerShell and in this example -Path is the parameter being tested $testfilepath is the argument value being passed. |
Sorry, something went wrong.
|
I agree with Sean Wheeler (@sdwheeler) here. Switch is definitely wrong. Reading the name of the test, it's referring to the Path parameter. |
Sorry, something went wrong.
|
Thank you Sean Wheeler (@sdwheeler) and Steve Lee (@SteveL-MSFT) for the feedback. I would have replied sooner but I got a little busy. |
Sorry, something went wrong.
|
Sean Wheeler (@sdwheeler) Please update your review. |
Sorry, something went wrong.
|
romero126 Thanks for your contribution! |
Sorry, something went wrong.
|
🎉v7.0.0-preview.5 has been released which incorporates this pull request.:tada: Handy links: |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
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 parameter instead of switch.
PR Checklist