-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Replace Ambiguous language for tests from switch to parameter for accuracy #10666
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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
parameter
instead ofswitch
.PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.