-
Notifications
You must be signed in to change notification settings - Fork 4k
AzureApiManagement breaking changes for api version 2019-12-01 #11618
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
AzureApiManagement breaking changes for api version 2019-12-01 #11618
Conversation
Can one of the admins verify this patch? |
...agement/ApiManagement.ServiceManagement/Commands/GetAzureApiManagementAuthorizationServer.cs
Show resolved
Hide resolved
Please rerecord the test cases. @vfedonkin |
@wyunchi-ms , tests rerecorded. |
I will retarget this PR to "generation" branch, as told in https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/545 |
@msJinLei , please advise, should we merge this PR into "generation" branch, or should we continue with the master? Generation branch doesn't have our ApiManagement code at all. |
@wyunchi-ms , tests rerecorded. Anything else required in this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @vfedonkin , @solankisamir there are still 33 case failed in ApiManagement. Could you please rerecord these test cases? |
@wyunchi-ms , found it. Tests rerecorded. |
add online version here like In this and other files Get-AzApiManagementAuthorizationServerClientSecret.md Online version in the header of the file is incorrect. The cor. Refers to: src/ApiManagement/ApiManagement/help/Get-AzApiManagementAuthorizationServerClientSecret.md:4 in 0c4a985. [](commit_id = 0c4a985, deletion_comment = False) |
@wyunchi-ms , @solankisamir , tests rerecorded, help fixed |
@solankisamir , @wyunchi-ms , all checks are green now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
Azure Api Management breaking changes for may build
Implementation of the changes described here: https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/545
Breaking change notification was included into april rc:
#11602
Checklist
CONTRIBUTING.md
ChangeLog.md
file(s) has been updated:ChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header -- no new version header should be added