Skip to content
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

Fix broken install link. #5780

Merged
merged 2 commits into from
Apr 9, 2018
Merged

Fix broken install link. #5780

merged 2 commits into from
Apr 9, 2018

Conversation

sdas-tech
Copy link

@sdas-tech sdas-tech commented Mar 12, 2018

Replace dead Azure CLI install link with a working one that points to the official docs.


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

General Guidelines

  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

  • Each command and parameter has a meaningful description.
  • Each new command has a test.

(see Authoring Command Modules)

Replace dead Azure CLI install link with a working one that points to the official docs.
@promptws
Copy link

View a preview at https://prompt.ws/r/Azure/azure-cli/5780
This is an experimental preview for @microsoft users.

Copy link
Member

@derekbekoe derekbekoe left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. One doc change.

@@ -42,7 +42,7 @@ using NPM, upgrade with `npm upgrade -g azure-cli`. If you used an installer,
we recommend downloading the latest installer to upgrade.

To install the Azure CLI 2.0, follow the steps for your preferred platform or
environment on our [Installation Guide](https://github.com/Azure/azure-cli/blob/master/README.rst#installation).
environment on our [Installation Guide](https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest).
Copy link
Member

Choose a reason for hiding this comment

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

Please remove ?view=azure-cli-latest from the URL.
Doc team would rather the server redirect to the appropriate view.

@sdas-tech
Copy link
Author

Updated the link as requested.

@derekbekoe derekbekoe merged this pull request into Azure:master Apr 9, 2018
@derekbekoe
Copy link
Member

@Shpetz The PR was made to the wrong branch. I have submitted a new PR with your commits to the correct dev branch.

@derekbekoe
Copy link
Member

#6054

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants