Skip to content
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

Fix service tags #125580

Merged
merged 1 commit into from
Feb 11, 2025
Merged

Fix service tags #125580

merged 1 commit into from
Feb 11, 2025

Conversation

jnysteen
Copy link
Contributor

The current documentation contains erroneous whitespaces - the proper tags are listed here: https://learn.microsoft.com/en-us/azure/virtual-network/service-tags-overview.

This PR removes the whitespaces.

Copy link
Contributor

@jnysteen : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit a9af42a:

✅ Validation status: passed

File Status Preview URL Details
includes/api-management-recommended-nsg-rules.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@ttorble
Copy link
Contributor

ttorble commented Jan 24, 2025

@dlepow

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Jan 24, 2025
Copy link
Contributor

@dlepow dlepow left a comment

Choose a reason for hiding this comment

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

Thanks @jnysteen for fixing those tags. LGTM. Much appreciated!

@dlepow
Copy link
Contributor

dlepow commented Feb 11, 2025

#sign-off

@ShannonLeavitt ShannonLeavitt merged commit 251fa75 into MicrosoftDocs:main Feb 11, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants