Skip to content

Add advisor to default install#4956

Merged
tjprescott merged 1 commit intoAzure:devfrom
derekbekoe:add-advisor
Nov 22, 2017
Merged

Add advisor to default install#4956
tjprescott merged 1 commit intoAzure:devfrom
derekbekoe:add-advisor

Conversation

@derekbekoe
Copy link
Copy Markdown
Member

No description provided.

@azuresdkci
Copy link
Copy Markdown
Contributor

View a preview at https://prompt.ws/r/Azure/azure-cli/4956
This is an experimental preview for @microsoft.com users.
(It may take a minute or two for your instance to be ready)
Email feedback to 'azfeedback' with subject 'Prompt Feedback'.

@tjprescott tjprescott merged commit 7ce5e53 into Azure:dev Nov 22, 2017
tjprescott added a commit that referenced this pull request Nov 27, 2017
* sp reset-credentials from dev

* rebase polish rbac error message

* rebase role assignment list: show default assignments for classic administrators

* Modified AzureUSGov Endpoint to .US (#4877)

* Modified AzureUSGov Endpoint to .US

* Updated History.RST to include USGovAADEndPoint Change

* Updated Setup.py to match HISTORY.RST

* Reduced previous change in HISTORY.RST to a single line

* HISTORY.rst merge with latest commit

* HISTORY.RST Syntax Correction

* webapp/functionapp: ensure list/show display correct set of apps (#4891)

* Fix authoring doc typo (#4903)

* Changing CLI for azure-consumption reflecting the new api-version (#4900)

* Changing CLI for azure-consumption reflecting the new api-version

* Modifying Setup.py and history.rst

* Implementing review comments

* Rebase remove component module

* [acr] typo fix in help (#4904)

* [acr] typo fix in help

* [acr] bump version and add history line item

* Rebase of support raw format on resource show

* Web: add node appsetting when runtime is not specified (#4907)

* webapp: fix a bug in the cert name generation (#4909)

* Fix container module default ports (#4950)

* fix default ports

* bump up version

* update history

* Advisor command module (#4898)

* Initial change to add Advisor module.

* Add list recommendations command.

* More commands.

* More commands.

* Pylint fixes and adding a test.

* More updates.

* More updates.

* Fixing precheck errors.

* Fixing more precheck errors.

* Initial change to add Advisor module.

* Add list recommendations command.

* More commands.

* More commands.

* Pylint fixes and adding a test.

* More updates.

* More updates.

* Fixing precheck errors.

* Fixing more precheck errors.

* Adding missing manifest.

* Fixes for test failures.

* Addressing review comments and add unit tests.

* More fixes.

* Address CI failures.

* Extra space.

* One more fix.

* rebase sql usage commands.

* Add advisor to default install (#4956)

* Add consumption module to code owners (#4906)

* Added sql server conn-policy show/update commands (#4888)

* Update proj file
@derekbekoe derekbekoe deleted the add-advisor branch November 27, 2017 18:48
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