fix: typo in docs/snapshot-analysis.md - #6508
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mzazrivec The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Pull request overview
Fixes a documentation typo in docs/snapshot-analysis.md so the AZURE_TOKEN_CREDENTIALS example matches the correct Azure Identity credential type name used in the codebase (e.g., azidentity.NewAzureCLICredential).
Changes:
- Correct
AzureCLICredentials→AzureCLICredentialin the snapshot analysis guidance.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
What
The correct value should be:
AZURE_TOKEN_CREDENTIALS=AzureCLICredentialWhy
For posterity.
Testing
Testing is required for feature completion and tests should be part of the pull
request along with the feature changes.
Describe the testing provided. If you did not add tests, provide a clear
justification.
Special notes for your reviewer
PR Checklist
If E2E tests are included:
demonstrate that the test is able to detect a defect/error and fail with
proper error message and logs which communicates nature of the problem.