Update network-requirements.md #7
Conversation
Not for merging just yet. Needs some additional rollouts and validation.
|
@edyoung : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 8150436:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| articles/azure-arc/servers/includes/network-requirements.md | Details |
articles/azure-arc/servers/includes/network-requirements.md
- Line 84, Column 73: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/azure/governance/machine-configuration/overview#network-requirements' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites. - Line 84, Column 73: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/azure/governance/machine-configuration/overview#network-requirements' will be broken in isolated environments. Replace with a relative link.
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.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
|
When this PR is marked ready for review, can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
|
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
Not for merging just yet. Needs some additional rollouts and validation.