Skip to content
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

Add-AzureRmAutomationModule and Set-AzureRmAutomationModule 'ContentLink' parameters should have the same name. #3775

Closed
lfshr opened this issue Apr 14, 2017 · 3 comments
Assignees
Labels
Automation Service Attention This issue is responsible by Azure service team.

Comments

@lfshr
Copy link

lfshr commented Apr 14, 2017

Cmdlet(s)

Add-AzureRmAutomationModule
Set-AzureRmAutomationModule

PowerShell Version

Name Value
PSVersion 5.1.14393.1066
PSEdition Desktop
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0...}
BuildVersion 10.0.14393.1066
CLRVersion 4.0.30319.42000
WSManStackVersion 3.0
PSRemotingProtocolVersion 2.3
SerializationVersion 1.1.0.1

Module Version

Name Version
AzureRM.Automation 2.8.0

OS Version

10.0.14393.1066

Description

Add-AzureRmAutomationModule currently has a parameter called 'ContentLink' and Set-AzureRmAutomationModule contains a parameter called 'ContentLinkUri' and 'ContentVersion'. I feel that this is unclear and inconsistent and one parameter name/set should be used. An alias could be added to avoid this being a breaking change.

@cormacpayne
Copy link
Member

@safeermohammed Hey Safeer, would you mind taking a look at this issue?

@safeermohammed
Copy link
Contributor

@vrdmr Lets try to get this change with your updates

@lfshr
Copy link
Author

lfshr commented Aug 31, 2017

Closed by #4349

@lfshr lfshr closed this as completed Aug 31, 2017
@bsiegel bsiegel added the Service Attention This issue is responsible by Azure service team. label Sep 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Automation Service Attention This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

6 participants