Skip to content

Conversation

@calvinsID
Copy link
Contributor

Description
Importing ResourceManagementClient incorrectly causes it to use latest version of python SDK automatically, instead of the version specified by core CLI

This checklist is used to make sure that common guidelines for a pull request are followed.

calvinsID added 2 commits May 19, 2021 08:37
…l cause ResourceManagementClient to use latest version of python SDK automatically, instead of using version controlled by core cli
@calvinsID calvinsID requested review from Juliehzl and qwordy as code owners May 19, 2021 16:06
@yonzhan yonzhan requested a review from zhoxing-ms May 20, 2021 01:34
@yonzhan yonzhan added this to the S188 milestone May 20, 2021
@yonzhan
Copy link
Collaborator

yonzhan commented May 20, 2021

AppService

@zhoxing-ms zhoxing-ms merged commit 051e30a into Azure:dev May 20, 2021
zhoxing-ms pushed a commit that referenced this pull request May 21, 2021
…GA (05/04) (#17869)

* Initial draft for formUiDefinition

* Tested with generated SDK from Azure/sdk-release-request#1469

* TemplateSpecsVersion parameters artifacts and template have been replaced by linked_templates and main_template

* Restored new lined at end of launch.json file

* reverting changes to setup.py until sdk release

* TemplateSpecs regression addressed #17733 (comment)

* Customer reported bug fix

* Removed _packing_engine changes

* Re-recorded test cases with template specs

* Reviewer suggestion

* Re-recorded template specs tests live

* Fixed CLI Style

* Merged with dev

* Removed merge conflicts

* Removed merge conflicts

* Restored new line for launch.json

* Restored new lines at end of files

* Bump template specs SDK to latest release

* Tested with new SDK

* Deployments Tests with new SDK

* SDK Bump to 18.0.0

* Tracking Linux requirements

* Merged with dev and added changes from #18142

* Rerecord test_nw_connection_monitor_v2_creation

Co-authored-by: Tarun Sunkaraneni <[email protected]>
Co-authored-by: Dania Etienne <[email protected]>
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.

5 participants