Fix bug in StaticAnalysis that threw when renaming a parameter #7675

cormacpayne · 2018-10-26T00:16:58Z

Description

Fix for #7662

I have read the Submitting Changes section of CONTRIBUTING.md
The title of the PR is clear and informative
The appropriate change log has been updated
The PR does not introduce breaking changes
If applicable, the changes made in the PR have proper test coverage
For public API changes to cmdlets:
- the changes have gone through a cmdlet design review (Microsoft internal partners only)
- the cmdlet markdown files were generated using the platyPS module

…ith aliases is added to a set

markcowl

Can we add an example of this to the unit tests for the breaking change analyzer (parameter replaced by an alias)

… alias

Fix bug in StaticAnalysis that threw when a new mandatory parameter w…

48de6dd

…ith aliases is added to a set

azuresdkci1x added the needs-review label Oct 26, 2018

markcowl requested changes Oct 26, 2018

View reviewed changes

markcowl added needs-revision and removed needs-review labels Oct 26, 2018

markcowl assigned cormacpayne Oct 26, 2018

Add test to ensure no exceptions are thrown for parameter replaced by…

7d4c2e7

… alias

cormacpayne assigned markcowl and unassigned cormacpayne Oct 26, 2018

cormacpayne added needs-review and removed needs-revision labels Oct 26, 2018

markcowl approved these changes Oct 29, 2018

View reviewed changes

markcowl merged commit ca62db5 into Azure:preview Oct 29, 2018

cormacpayne deleted the renamed-param-fix branch January 16, 2019 21:34