Skip to content

Conversation

@mikefrobbins
Copy link
Contributor

@mikefrobbins mikefrobbins commented Nov 17, 2025

PR Summary

Fixes AB#509594, Fixes #5533 - Updated linux script install article to use pip.

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.

@learn-build-service-prod

This comment was marked as outdated.

Copilot finished reviewing on behalf of mikefrobbins November 17, 2025 22:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the Linux installation documentation for Azure CLI by replacing the legacy script-based installation method with a pip-based approach using Python virtual environments. The update improves installation reliability, adds security warnings about hash-based integrity checks, and simplifies the update and uninstall processes.

Key Changes

  • Replaced curl-based install script with pip installation in a virtual environment
  • Updated Python requirements from 3.8-3.10 to 3.10+ and removed libffi/OpenSSL dependencies
  • Simplified troubleshooting section by removing script-specific issues and focusing on common PATH/proxy problems

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ln -sf ~/lib/azure-cli/bin/az ~/bin/az

# Ensure ~/bin is on PATH for future shells
grep -q 'export PATH="$HOME/bin:$PATH"' ~/.bashrc || echo 'export PATH="$HOME/bin:$PATH"' >> ~/.bashrc
Copy link

Copilot AI Nov 17, 2025

Choose a reason for hiding this comment

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

In bash, the grep -q command with the || operator may not behave as expected if .bashrc doesn't exist. While echo will create the file, a safer approach would be to ensure the file exists first, or use:

grep -qxF 'export PATH="$HOME/bin:$PATH"' ~/.bashrc 2>/dev/null || echo 'export PATH="$HOME/bin:$PATH"' >> ~/.bashrc

The -x flag ensures an exact line match, -F treats the pattern as a fixed string (not regex), and 2>/dev/null suppresses error output if the file doesn't exist.

Suggested change
grep -q 'export PATH="$HOME/bin:$PATH"' ~/.bashrc || echo 'export PATH="$HOME/bin:$PATH"' >> ~/.bashrc
grep -qxF 'export PATH="$HOME/bin:$PATH"' ~/.bashrc 2>/dev/null || echo 'export PATH="$HOME/bin:$PATH"' >> ~/.bashrc

Copilot uses AI. Check for mistakes.
@learn-build-service-prod

This comment was marked as outdated.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 771d2e0:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/Latest-version/includes/cli-install-linux-script.md ✅Succeeded View (azure-cli-latest)

For more details, please refer to the build report.

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.

Feedback

1 participant