-
Notifications
You must be signed in to change notification settings - Fork 146
Fixed missing link to az aks mesh get-revisions #338
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
base: main
Are you sure you want to change the base?
Conversation
|
@jovieir : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 0a3c544:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| articles/aks/istio-support-policy.md | Details |
articles/aks/istio-support-policy.md
- Line 59, Column 77: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/cli/azure/aks/mesh?view=azure-cli-latest#az-aks-mesh-get-revisions' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites. - Line 59, Column 77: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/cli/azure/aks/mesh?view=azure-cli-latest#az-aks-mesh-get-revisions' will be broken in isolated environments. Replace with a relative link. - Line 59, Column 77: [Suggestion: preserve-view-not-set - See documentation]
You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/en-us/cli/azure/aks/mesh?view=azure-cli-latest#az-aks-mesh-get-revisions
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
There was a problem hiding this 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 fixes broken formatting by adding a missing external link reference for the az aks mesh get-revisions command that was previously referenced but not defined in the document's link section.
- Added the
az-aks-mesh-get-revisionslink definition to the external links section
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
articles/aks/istio-support-policy.md
Outdated
| [aks-release-notes]: https://github.com/Azure/AKS/releases | ||
| [aks-release-status]: https://releases.aks.azure.com/ | ||
| [istio-support-calendar]: https://istio.io/latest/docs/releases/supported-releases/#support-status-of-istio-releases | ||
| [az-aks-mesh-get-revisions]: https://learn.microsoft.com/en-us/cli/azure/aks/mesh?view=azure-cli-latest#az-aks-mesh-get-revisions |
Copilot
AI
Nov 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing whitespace detected at the end of the line. Please remove the trailing space after the URL.
| [az-aks-mesh-get-revisions]: https://learn.microsoft.com/en-us/cli/azure/aks/mesh?view=azure-cli-latest#az-aks-mesh-get-revisions | |
| [az-aks-mesh-get-revisions]: https://learn.microsoft.com/en-us/cli/azure/aks/mesh?view=azure-cli-latest#az-aks-mesh-get-revisions |
|
@jovieir can you remove Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
|
Learn Build status updates of commit 3860c30:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| articles/aks/istio-support-policy.md | Details |
articles/aks/istio-support-policy.md
- Line 59, Column 77: [Warning: file-not-found - See documentation]
Invalid file link: './cli/azure/aks/mesh'.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
|
Learn Build status updates of commit d7f2e26: ✅ Validation status: passed
For more details, please refer to the build report. |
|
@ttorble thanks for the heads up! Just cleaned it up |
Added external link to fix this broken formatting: