{"action":"created","issue":{"url":"https://api.github.com/repos/PowerShell/DscResources/issues/98","repository_url":"https://api.github.com/repos/PowerShell/DscResources","labels_url":"https://api.github.com/repos/PowerShell/DscResources/issues/98/labels{/name}","comments_url":"https://api.github.com/repos/PowerShell/DscResources/issues/98/comments","events_url":"https://api.github.com/repos/PowerShell/DscResources/issues/98/events","html_url":"https://github.com/PowerShell/DscResources/issues/98","id":132056934,"number":98,"title":"Add a guideline that recommends against using Positional Parameters in cmdlets","user":{"login":"PlagueHO","id":7589164,"avatar_url":"https://avatars.githubusercontent.com/u/7589164?v=3","gravatar_id":"","url":"https://api.github.com/users/PlagueHO","html_url":"https://github.com/PlagueHO","followers_url":"https://api.github.com/users/PlagueHO/followers","following_url":"https://api.github.com/users/PlagueHO/following{/other_user}","gists_url":"https://api.github.com/users/PlagueHO/gists{/gist_id}","starred_url":"https://api.github.com/users/PlagueHO/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PlagueHO/subscriptions","organizations_url":"https://api.github.com/users/PlagueHO/orgs","repos_url":"https://api.github.com/users/PlagueHO/repos","events_url":"https://api.github.com/users/PlagueHO/events{/privacy}","received_events_url":"https://api.github.com/users/PlagueHO/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/PowerShell/DscResources/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-02-08T05:38:04Z","updated_at":"2016-02-09T18:28:40Z","closed_at":null,"body":"Using positional parameters instead of named parameters in most cases reduces code readability and is recommended against in most cases (e.g. https://blogs.technet.microsoft.com/heyscriptingguy/2012/04/22/the-problem-with-powershell-positional-parameters/).\r\n\r\nI did not see a policy in the StyleGuidelines.md document against using positional parameters, but I think there should be one - e.g. don't use positional parameters (always use named parameters). PSScriptAnalyzer will also throw warnings against against this as well.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/PowerShell/DscResources/issues/comments/181993091","html_url":"https://github.com/PowerShell/DscResources/issues/98#issuecomment-181993091","issue_url":"https://api.github.com/repos/PowerShell/DscResources/issues/98","id":181993091,"user":{"login":"KarolKaczmarek","id":6466574,"avatar_url":"https://avatars.githubusercontent.com/u/6466574?v=3","gravatar_id":"","url":"https://api.github.com/users/KarolKaczmarek","html_url":"https://github.com/KarolKaczmarek","followers_url":"https://api.github.com/users/KarolKaczmarek/followers","following_url":"https://api.github.com/users/KarolKaczmarek/following{/other_user}","gists_url":"https://api.github.com/users/KarolKaczmarek/gists{/gist_id}","starred_url":"https://api.github.com/users/KarolKaczmarek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KarolKaczmarek/subscriptions","organizations_url":"https://api.github.com/users/KarolKaczmarek/orgs","repos_url":"https://api.github.com/users/KarolKaczmarek/repos","events_url":"https://api.github.com/users/KarolKaczmarek/events{/privacy}","received_events_url":"https://api.github.com/users/KarolKaczmarek/received_events","type":"User","site_admin":false},"created_at":"2016-02-09T18:28:40Z","updated_at":"2016-02-09T18:28:40Z","body":"True, that will also allow us to enable PSSA runs on all other repos where they are currently disabled without moving them to WMF5 test VMs"}}