Skip to content

Update pre-post-prerequisites.md #124487

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

wjpigott
Copy link
Contributor

Added better example of IaM role needed or Custom Role

Added better example of IaM role needed or Custom Role
Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 7da5aef:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/update-manager/includes/pre-post-prerequisites.md ⚠️Warning Details

articles/update-manager/includes/pre-post-prerequisites.md

  • Line 24, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
  • Line 28, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).

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:

@Jak-MS
Copy link
Contributor

Jak-MS commented Sep 23, 2024

@SnehaSudhirG
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 Sep 23, 2024
Copy link
Contributor

Learn Build status updates of commit ddc04ae:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/update-manager/includes/pre-post-prerequisites.md ⚠️Warning Details

articles/update-manager/includes/pre-post-prerequisites.md

  • Line 24, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
  • Line 28, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).

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:

3. Import the `Az.ResourceGraph` module, ensure the module is updated to ThreadJob with the module version 2.0.3.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

actually this would be better code to use.
#Install the Resource Graph module from PowerShell Gallery
if (-not (Get-Module -Name Az.ResourceGraph -ListAvailable)) {
Install-Module -Name Az.ResourceGraph -Force
}

@wjpigott
Copy link
Contributor Author

@SnehaSudhirG any updates on this merge?

@wjpigott
Copy link
Contributor Author

@Jak-MS any updates on this article getting changed? Thanks!

@Jak-MS
Copy link
Contributor

Jak-MS commented Oct 31, 2024

@SnehaSudhirG - nudge, can you review this? thanks!

@v-ccolin
Copy link
Contributor

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

@SnehaSudhirG
Copy link
Contributor

@wjpigott - Thank you for your patience. Awaiting confirmation on this update. Will confirm on this soon.

@v-ccolin - Thanks for the follow-up Will update shortly.

@v-dirichards
Copy link
Contributor

@SnehaSudhirG Have you received confirmation yet? Thanks.

@v-dirichards
Copy link
Contributor

@SnehaSudhirG Can you review this old PR and determine whether it needs to be closed or merged?

@MicrosoftDocs/public-repo-pr-review-team

@ttorble
Copy link
Contributor

ttorble commented Feb 17, 2025

I sent email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

@ttorble
Copy link
Contributor

ttorble commented Mar 19, 2025

I sent a Teams message to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

@ttorble
Copy link
Contributor

ttorble commented Mar 24, 2025

Content owner has updated to say that confirmation is needed for these changes, still in progress and actively monitored.

@v-ccolin
Copy link
Contributor

v-ccolin commented May 1, 2025

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aq-pr-triaged tracking label for the PR review team Change sent to author do-not-merge
6 participants