Skip to content

2: Rename Clients DLLs #101

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Dec 6, 2018
Merged

2: Rename Clients DLLs #101

merged 4 commits into from
Dec 6, 2018

Conversation

MiYanni
Copy link
Contributor

@MiYanni MiYanni commented Dec 5, 2018

This does the Common changes for Azure/azure-powershell#7374.

Only changed the DLL names and package names for Clients. This does not include Storage (does include Storage.Management).

@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 5, 2018

sign: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/ps-common-sign/232/

@MiYanni MiYanni changed the title Rename Clients DLLs 2: Rename Clients DLLs Dec 5, 2018
@MiYanni
Copy link
Contributor Author

MiYanni commented Dec 5, 2018

sign: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/ps-common-sign/233/

markcowl
markcowl requested changes Dec 6, 2018
View reviewed changes
Copy link
Member

@markcowl markcowl left a comment

Choose a reason for hiding this comment

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

Need to update the previous PR and pull it onto this one. The changes specific to this PR are OK

@MiYanni MiYanni mentioned this pull request Dec 6, 2018
Merged
markcowl
markcowl requested changes Dec 6, 2018
View reviewed changes
@markcowl markcowl merged commit 6962e09 into Azure:master Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markcowl markcowl markcowl approved these changes

Assignees
No one assigned
Labels
Waiting for CI :shipit:
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MiYanni @markcowl