Skip to content

Conversation

@hovsepm
Copy link
Contributor

@hovsepm hovsepm commented Feb 5, 2016

No description provided.

}
}

function Uninstall-ModuleWithVersionCheck([string]$Name,[string]$MinVersion)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may decide that we need an uninstall cmdlet - currently, if you uninstall the manifest module, the dependent modules remain. We may retain an 'Uninstall-AzureRm' script which uninstalls the module and its dependencies (not sure if it's possible for a cmdlet to remove all associated modules. Regardless, we should decide on some acceptance criteria before merging.

@hovsepm hovsepm changed the title [#112641071] Convert AzureRM to a 'shell' module with version range dependencies [Do not merge] [#112641071] Convert AzureRM to a 'shell' module with version range dependencies Feb 23, 2016
@hovsepm
Copy link
Contributor Author

hovsepm commented Mar 2, 2016

@hovsepm
Copy link
Contributor Author

hovsepm commented Mar 2, 2016

@hovsepm
Copy link
Contributor Author

hovsepm commented Mar 7, 2016

@hovsepm
Copy link
Contributor Author

hovsepm commented Mar 8, 2016

@hovsepm hovsepm changed the title [Do not merge] [#112641071] Convert AzureRM to a 'shell' module with version range dependencies [#112641071] Convert AzureRM to a 'shell' module with version range dependencies Mar 9, 2016
@hovsepm
Copy link
Contributor Author

hovsepm commented Mar 10, 2016

@hovsepm
Copy link
Contributor Author

hovsepm commented Mar 11, 2016

@hovsepm
Copy link
Contributor Author

hovsepm commented Mar 11, 2016

hovsepm pushed a commit that referenced this pull request Mar 11, 2016
[#112641071] Convert AzureRM to a 'shell' module with version range dependencies
@hovsepm hovsepm merged commit ed2e651 into Azure:dev Mar 11, 2016
@hovsepm hovsepm deleted the AzureRmShell branch March 11, 2016 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants