-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Since all of the following are default values I don't see the need to name them for every parameter. In my opinion It makes it more readable to not specify them in this case.
PS C:\> [parameter]::new()
Position : -2147483648
ParameterSetName : __AllParameterSets
Mandatory : False
ValueFromPipeline : False
ValueFromPipelineByPropertyName : False
ValueFromRemainingArguments : False
HelpMessage :
HelpMessageBaseName :
HelpMessageResourceId :
DontShow : False
TypeId : System.Management.Automation.ParameterAttribute
I've created a PR to this end. I kept Mandatory even if the default of $false was used
Metadata
Metadata
Assignees
Labels
No labels