Skip to content

Clarify --auth-method exception placement for ACR commands#2944

Merged
yunjchoi merged 3 commits into
mainfrom
copilot/doc-gap-fix-acr-options
Jun 26, 2026
Merged

Clarify --auth-method exception placement for ACR commands#2944
yunjchoi merged 3 commits into
mainfrom
copilot/doc-gap-fix-acr-options

Conversation

Copilot AI commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

The CLI reference still implied --auth-method applied uniformly, even though acr registry list and acr registry repository list no longer support it. This updates the documentation so the exception is described where ACR commands are documented, matching the section-local pattern already used elsewhere.

  • Documentation correction

    • Removed the ACR-specific --auth-method exception note from the Global Options section.
    • Added the note under Azure Container Registry (ACR) Operations, so the exception is visible at the point of use.
  • Scope alignment

    • Kept the Global Options section generic (available for most commands) and moved command-specific behavior into the relevant command section.
    • Mirrors the existing section-local note style used for cloudarchitect design.
### Azure Container Registry (ACR) Operations

> [!NOTE]
> The `acr registry list` and `acr registry repository list` commands do not support `--auth-method`.

Invoking Livetests

Copilot submitted PRs are not trustworthy by default. Users with write access to the repo need to validate the contents of this PR before leaving a comment with the text /azp run mcp - pullrequest - live. This will trigger the necessary livetest workflows to complete required validation.

Copilot AI changed the title [WIP] Fix documentation for ACR registry options Clarify --auth-method exceptions in Azure MCP global options docs Jun 24, 2026
Copilot AI requested a review from yunjchoi June 24, 2026 20:45
Copilot AI changed the title Clarify --auth-method exceptions in Azure MCP global options docs Clarify --auth-method exception placement for ACR commands Jun 24, 2026
@yunjchoi yunjchoi requested a review from msalaman June 24, 2026 23:48
@yunjchoi yunjchoi marked this pull request as ready for review June 26, 2026 18:19
@yunjchoi yunjchoi requested review from a team as code owners June 26, 2026 18:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Azure MCP CLI command reference to clarify that --auth-method is not supported by two ACR commands, and places that exception note within the ACR section (instead of the Global Options section) to match the section-local documentation pattern used elsewhere in the document.

Changes:

  • Added a section-local NOTE under Azure Container Registry (ACR) Operations explaining that acr registry list and acr registry repository list do not support --auth-method.
  • Kept Global Options wording generic (“available for most commands”) by moving command-specific behavior to the command’s documentation section.

@github-project-automation github-project-automation Bot moved this from Untriaged to In Progress in Azure MCP Server Jun 26, 2026
@yunjchoi yunjchoi merged commit 201470b into main Jun 26, 2026
25 checks passed
@yunjchoi yunjchoi deleted the copilot/doc-gap-fix-acr-options branch June 26, 2026 19:39
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants