Skip to content

{AKS} Correct indefinite article in Azure Dev Spaces docstring - #33885

Merged
Cooper Cox (coopercox-ms) merged 1 commit into
Azure:devfrom
Hashim1999164:fix/acs-custom-a-azure-27937
Aug 21, 2026
Merged

{AKS} Correct indefinite article in Azure Dev Spaces docstring#33885
Cooper Cox (coopercox-ms) merged 1 commit into
Azure:devfrom
Hashim1999164:fix/acs-custom-a-azure-27937

Conversation

@Hashim1999164

@Hashim1999164 Hashim Khan (Hashim1999164) commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

🤖 PR Validation — ️✔️ All clear

Breaking Changes Tests
️✔️ None ️✔️ 130/130

Summary

Updates the ads_use_dev_spaces docstring in ACS custom.py from "a Azure Dev Spaces controller" to "an Azure Dev Spaces controller".

Related to #27937

Test plan

  • Confirmed the docstring now uses "an Azure"
  • Spot check related ACS Dev Spaces help text if desired

The ads_use_dev_spaces parameter docs said "a Azure Dev Spaces controller". Use "an Azure" to match standard article usage.

Fixes Azure#27937
Copilot AI lite review requested due to automatic review settings August 13, 2026 00:34
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

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 a minor grammar issue in the ACS aks_use_dev_spaces command docstring to use the correct indefinite article before “Azure”, improving the CLI help text and addressing issue #27937.

Changes:

  • Changed “a Azure Dev Spaces controller” to “an Azure Dev Spaces controller” in the aks_use_dev_spaces docstring.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@yonzhan

Copy link
Copy Markdown
Collaborator

ACS

@x-engineering-agent

Copy link
Copy Markdown
Contributor

Live test skipped

⏭️ Skipping the live test for this revision because no new test file was added (tests/**/test_*.py).

The live-test pipeline runs only the test files a PR changes, so there is nothing to execute for this commit. This is informational — a regression test is encouraged where it makes sense, but not required. If a test file is added in a later commit, the live test will run automatically.

@x-engineering-agent x-engineering-agent Bot 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.

CI/Live-Test Summary

Live test: Skipped — this PR changes no test files (docstring-only fix), so azdev test --live has nothing to run.

Upstream CI: ✅ All 1 check passed.

No further action needed from automation here. 👍

@coopercox-ms Cooper Cox (coopercox-ms) changed the title Fix a Azure article in ACS Dev Spaces docstring {ACS} Fix #27937: Correct indefinite article in Azure Dev Spaces docstring Aug 20, 2026
@coopercox-ms

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).

@coopercox-ms Cooper Cox (coopercox-ms) changed the title {ACS} Fix #27937: Correct indefinite article in Azure Dev Spaces docstring {ACS} Correct indefinite article in Azure Dev Spaces docstring Aug 20, 2026
@coopercox-ms Cooper Cox (coopercox-ms) changed the title {ACS} Correct indefinite article in Azure Dev Spaces docstring {AKS} Correct indefinite article in Azure Dev Spaces docstring Aug 21, 2026
@coopercox-ms

Copy link
Copy Markdown
Member

FumingZhang can we please get your review on this PR. Thanks!

@FumingZhang

Copy link
Copy Markdown
Member

FumingZhang can we please get your review on this PR. Thanks!

sure, approved

@coopercox-ms
Cooper Cox (coopercox-ms) merged commit ecf236f into Azure:dev Aug 21, 2026
73 checks passed
@a0x1ab Aditya Pujara (a0x1ab) added X Engineering Agent Reviewed Pull request reviewed by X Engineering Agent and removed azure-client-tools-agent labels Sep 2, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown
Contributor

🔔 Routing this PR to @Azure/act-observability-squad.

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

Labels

act-observability-squad AKS az aks/acs/openshift Auto-Assign Auto assign by bot customer-reported Issues that are reported by GitHub users external to the Azure organization. X Engineering Agent Reviewed Pull request reviewed by X Engineering Agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants