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

Correct export path in example #3541

Merged
merged 3 commits into from
Oct 9, 2023
Merged

Correct export path in example #3541

merged 3 commits into from
Oct 9, 2023

Conversation

outorted
Copy link
Contributor

@outorted outorted commented Jul 5, 2023

The example is to create an HTML report, but the path of the report had the extension .xml. For an HTML report, this should be HTML

PR Summary

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

The example is to create an HTML report, but the path of the report had the extension .xml. For an HTML report, this should be HTML
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 8d87d8d:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/grouppolicy/Get-GPResultantSetOfPolicy.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@scanum
Copy link

scanum commented Oct 9, 2023

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 2e43bde:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/grouppolicy/Get-GPResultantSetOfPolicy.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@tiburd
Copy link
Contributor

tiburd commented Oct 9, 2023

@outorted
Hello - thank you for your contribution.
Before we can merge this pull request, the PR Checklist must be complete:
image

After you complete the checklist, please @ mention me so that I can merge the pull request.

Thank you!

@outorted
Copy link
Contributor Author

outorted commented Oct 9, 2023

@tiburd sorry, I've not done this before... I just ticked the boxes in the previous comment. I hope that's OK. If you need anything else from me, then please let me know.

@tiburd
Copy link
Contributor

tiburd commented Oct 9, 2023

@tiburd sorry, I've not done this before... I just ticked the boxes in the previous comment. I hope that's OK. If you need anything else from me, then please let me know.

@outorted
No worries at all!
I'll merge the pull request after it passes all the required (system-type) build checks....
Thanks!

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 7625a7e:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/grouppolicy/Get-GPResultantSetOfPolicy.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@tiburd tiburd merged commit d5bb15e into MicrosoftDocs:main Oct 9, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants